-
Notifications
You must be signed in to change notification settings - Fork 5
fix(deps): update angular monorepo to v20 (major) #1612
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
Conversation
|
eaf5d90 to
9449d68
Compare
4d68d41 to
d3a3168
Compare
2df4229 to
de156d4
Compare
acd2085 to
539d23e
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
59a67ef to
7629e94
Compare
ManuelMoeri
left a comment
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.
Looks good overall, only a handful of minor things I found. Frontend seems to be good as well.
However do pay attention to your commit messages. Per coincidence I spotted that you commit messages do not have the issue number appended to them.
frontend/src/app/components/key-result-type/key-result-type.component.ts
Show resolved
Hide resolved
7629e94 to
75674a3
Compare
MasterEvarior
left a comment
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.
Needs some minor adjustments but overall good job 🥳
frontend/src/app/components/checkin/check-in-form/check-in-form.component.ts
Outdated
Show resolved
Hide resolved
frontend/src/app/shared/application-page/application-page.component.ts
Outdated
Show resolved
Hide resolved
frontend/src/app/team-management/search-team-management/search-team-management.component.ts
Outdated
Show resolved
Hide resolved
frontend/src/app/team-management/team-list/team-list.component.ts
Outdated
Show resolved
Hide resolved
...app/team-management/team-management-mobile-filter/team-management-mobile-filter.component.ts
Outdated
Show resolved
Hide resolved
MasterEvarior
left a comment
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.
Found another small thing 🤓
frontend/src/app/team-management/team-list/team-list.component.ts
Outdated
Show resolved
Hide resolved
d0555c4 to
f6c5284
Compare
f6c5284 to
2ad5ff4
Compare
This PR contains the following updates:
^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0^19.0.3->^20.0.0Warning
Some dependencies could not be looked up. Check the Dependency Dashboard for more information.
Release Notes
angular/angular (@angular/animations)
v20.1.6Compare Source
v20.1.5Compare Source
compiler-cli
core
fetchresponses when loading external resources in JIT (#62992)http
set()inHttpResource(#62873)migrations
v20.1.4Compare Source
compiler
core
http
v20.1.3Compare Source
core
platform-browser
v20.1.2Compare Source
compiler
v20.1.1Compare Source
compiler
compiler-cli
core
http
Angular detected that aHttpClientrequest with thekeepaliveoption was sent using XHRwhen option is not true (#62536)router
v20.1.0Compare Source
common
compiler
compiler-cli
core
destroyedproperty onDestroyRef(#61849)destroyedproperty toEnvironmentInjector(#61951)forms
FormGroup/FormRecord(#55860)http
language-service
router
loadComponentandloadChildrenfunctions in the route's injection context (#62133)service-worker
v20.0.7Compare Source
compiler-cli
v20.0.6Compare Source
v20.0.5Compare Source
compiler-cli
core
ErrorHandlerfrom a destroyed injector (#61886)router
upgrade
v20.0.4Compare Source
core
APP_IDbefore injector is destroyed (#61885)migrations
v20.0.3Compare Source
v20.0.2Compare Source
core
onDestroyinoutputToObservable(#61882)v20.0.1Compare Source
compiler
core
takeUntilDestroyedcompletes immediately if DestroyRef already destroyed (#61847)onDestroyinResourceImplwhendestroy()is called (#61870)onDestroywhen observable errors intoSignal(#61596)migrations
service-worker
v20.0.0Compare Source
Blog post: https://blog.angular.dev/announcing-angular-v20-b5c9c06cf301
Breaking Changes
common
Yformatter (week-numbering year) without also includingw(week number) is now detected as suspicious date pattern, asyis typically intended.AsyncPipenow directly catches unhandled errors insubscriptions and promises and reports them to the application's
ErrorHandler. For Zone-based applications, these errors would havebeen caught by ZoneJS and reported to
ErrorHandlerso the result isgenerally the same. The change to the exact mechanism for reporting can
result in differences in test environments that will require test
updates.
compiler
'in' in an expression now refers to the operator
voidin an expression now refers to the operatorPreviously an expression in the template like
{{void}}referred to aproperty on the component class. After this change it now refers to the
voidoperator, which would make the above example invalid. If you haveexisting expressions that need to refer to a property named
void,change the expression to use
this.voidinstead:{{this.void}}.Parenthesis are always respected.
This can lead to runtime breakages when a nullish coalescing operator is nested within parentheses.
eg.
{{ (foo?.bar).baz }}will throw iffoois nullish. This is the same behavior as native JavaScript.core
TypeScript versions less than 5.8 are no longer supported.
the
TestBed.flushEffects()was removed - usethe
TestBed.tick()instead.provideExperimentalCheckNoChangesForDebughas severalbreaking changes:
provideCheckNoChangesConfiguseNgZoneOnStableoption is removed. This wasn't found to be generallymore useful than
intervalprovideExperimentalZonelessChangeDetectionisrenamed to
provideZonelessChangeDetectionas it is now "DeveloperPreview" rather than "Experimental".
InjectFlagshas been removed.injectno longer acceptsInjectFlags.Injector.getno longer acceptsInjectFlags.EnvironmentInjector.getno longer acceptsInjectFlags.TestBed.getno longer acceptsInjectFlags.TestBed.injectno longer acceptsInjectFlags.TestBed.gethas been removed. UseTestBed.injectinstead.afterRender was renamed to afterEveryRender.
Before upgrading to Angular v20, ensure the Node.js version is at least 20.11.1.
For the full list of supported versions, visit: https://angular.dev/reference/versions
PendingTasks.runno longer returns the result of theasync function. If this behavior is desired, it can be re-implemented
manually with the
PendingTasks.add. Be aware, however, that promise rejectionswill need to be handled or they can cause the node process to shut down
when using SSR.
Uncaught errors in listeners which were previously only reported to
ErrorHandlerare now also reported to Angular's internal errorhandling machinery. For tests, this means that the error will be
rethrown by default rather than only logging the error. Developers
should fix these errors, catch them in the test if the test is
intentionally covering an error case, or use
rethrowApplicationErrors: falseinconfigureTestingModuleas a last resort.The
anyoverload has been removed frominjector.get. It now only supportsProviderToken<T>and (deprecatedsince v4)
string.Animations are guaranteed to be flushed when Angular
runs automatic change detection or manual calls to
ApplicationRef.tick.Prior to this change, animations would not be flushed in some situations
if change detection did not run on any views attached to the
application. This change can affect tests which may rely on the old
behavior, often by making assertions on DOM elements that should have
been removed but weren't because DOM removal is delayed until animations
are flushed.
ApplicationRef.tickwill no longer catch and reporterrors to the appplication
ErrorHandler. Errors will instead be thrown out ofthe method and will allow callers to determine how to handle these
errors, such as aborting follow-up work or reporting the error and
continuing.
This commit deprecates
ng-reflect-*attributes and updates the runtime to stop producing them by default. Please refactor application and test code to avoid relying onng-reflect-*attributes.To enable a more seamless upgrade to v20, we've added the
provideNgReflectAttributes()function (can be imported from the@angular/corepackage), which enables the mode in which Angular would be producing those attribites (in dev mode only). You can add theprovideNgReflectAttributes()function to the list of providers within the bootstrap call.router
RedirectFncan now returnObservableorPromise. Any code that directly calls functions returning this typemay need to be adjusted to account for this.
required writable arrays have now been updated to accept readonly
arrays when no mutations are done.
Routeno longer includeanyinthe type union. The union includes functions for the functional guards
as well as a type matching
Injector.get:ProviderToken<T>|string.Note that string is still deprecated on both the route guards and
Injector.get.Deprecations
core
ngIf/ngFor/ngSwitchare deprecated. Use the control flow blocks instead (@for/@if/@switch).platform-browser
@angular/platform-browser-dynamicplatform-server
@angular/platform-server/testingUse e2e tests to verify SSR behavior instead.
common
updateLatestValueif view is destroyed before promise resolves (#58041)NgOptimizedImageexceeds the preload limit (#60879)compiler
@forblocks (#60495)inkeyword in Binary expression (#58432)compiler-cli
core
Injector.destroyonInjectorcreated withInjector.create(#60054)toObservableas stable (#60449)provideExperimentalCheckNoChangesForDebugtoprovideCheckNoChangesConfig(#60906)PendingTasks.run(#60044)withEventReplay()is invoked (#61077)@Attributeas optional (#60916)zone.jsas an optional peer dependency (#61616)hasPendingTasksobservers (#59723)forceRootflag for effects ([#60535](https://redirect.github.com/angular/angular/pulConfiguration
📅 Schedule: Branch creation - Between 01:00 AM and 05:59 AM, Monday through Friday ( * 1-5 * * 1-5 ) in timezone Europe/Zurich, Automerge - Between 05:00 AM and 07:59 AM, Monday through Friday ( * 5-7 * * 1-5 ) in timezone Europe/Zurich.
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.