-
Notifications
You must be signed in to change notification settings - Fork 2
Angular SSR + Shopularity Sample Project #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
@@ -0,0 +1,35 @@ | |||
import {environment} from "../../environments/environment"; | |||
|
|||
export function absolutizeRelativeUrls(html: string): string { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could not understand where would we use this util.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed unnecessary util
import {inject, Injectable} from "@angular/core"; | ||
import {Rest, RestService} from "@abp/ng.core"; | ||
import {PagedProducts, ProductWithCategory} from "../models"; | ||
import ResponseType = Rest.ResponseType; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could remove this import as it does not seem being used
import ResponseType = Rest.ResponseType;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
import {Pipe, PipeTransform} from "@angular/core"; | ||
import {environment} from "../../environments/environment"; | ||
|
||
@Pipe({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could not see the usage for this pipe, so maybe we could delete this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @erdemcaygor thank you for putting an effort on creating this sample. Other than the comments I left, it is mergable 🙌🏻
We could also check these components using RouterModule
import as it seems redundant.
-
Shopularity/angular/public-site/src/app/containers/product-list/product-list-item/product-list-item.component.ts
-
Shopularity/angular/public-site/src/app/containers/product-list/product-list.component.ts
-
Shopularity/angular/public-site/src/app/containers/product-detail/product-detail.component.ts
Resolves https://github.com/volosoft/vs-internal/issues/7321