-
Notifications
You must be signed in to change notification settings - Fork 6
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
CFWEB-177 Update Styleguide app to Angular 18 #23
base: main
Are you sure you want to change the base?
Conversation
-includes update to TypeScript 5.4 - includes update to new Angular build system - includes update to @angular-eslint/schematics@rc-v18 (final not ready yet)
The triggered build does fail on this pull request, this happens because of outdated configurations for wdio, which will be fixed in this PR. |
…5-Angular-18-Update # Conflicts: # package-lock.json
imports: [], | ||
providers: [ | ||
RecommendationService, | ||
provideHttpClient(withInterceptorsFromDi()), |
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 found this on the official Angular page for "withInterceptorsFromDi".
https://angular.dev/api/common/http/withInterceptorsFromDi?tab=description
Prefer withInterceptors and functional interceptors instead, as support for DI-provided interceptors may be phased out in a later release.
Maybe you should check 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.
Added a comment
Includes: