You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Angular apps created by the CLI are now zoneless (Angular 21 default).
add-layout: detects zone-based projects and renders provideZoneChangeDetection in app.config.ts from the template; zone.js dependency/polyfill are added only for them
Template services are providedIn: 'root', so app.config.ts only registers the router
Layout and auth templates use signals
@angular/cdk version derived from the project's @angular/core; globalNgCliVersion removed
e2e: verify generated apps contain no zone.js
Merge and release after devextreme@26.2 is released.
The reason will be displayed to describe this comment to others. Learn more.
🟡 Changes recommended
The provider merge can generate invalid router syntax, override custom authentication, and insert required services into the wrong configuration object.
Provider matching ignores import aliases: AuthService as AppAuth with { provide: AppAuth, useClass: CustomAuthService } is treated as missing AuthService, so the merge appends a default registration for an already configured token. The same issue affects an aliased provideRouter. Resolve imported symbols when comparing providers, and reuse their local bindings when inserting references.
The reason will be displayed to describe this comment to others. Learn more.
🔵 Needs a closer look
Angular release-candidate versions now select stable-only CDK ranges, causing dependency installation to fail before the corresponding stable CDK release is available.
Reverts 6dbfeab, 8cd6e68, d9fcabd, 0dbe244, 7e244c8.
The team decided to keep overwriting app.config.ts with the template.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The reason will be displayed to describe this comment to others. Learn more.
i've added for our services @Injectable({ providedIn: 'root' }). So here no reason to left them in providers array. It's debatable decision.
Leaving them in providers array is more transparent choice and matches our previous generated apps. But it seems @Injectable({ providedIn: 'root' }) is current angular convention because ng generate service does the same thing.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Angular apps created by the CLI are now zoneless (Angular 21 default).
add-layout: detects zone-based projects and rendersprovideZoneChangeDetectioninapp.config.tsfrom the template;zone.jsdependency/polyfill are added only for themprovidedIn: 'root', soapp.config.tsonly registers the router@angular/cdkversion derived from the project's@angular/core;globalNgCliVersionremovedMerge and release after devextreme@26.2 is released.