Skip to content

feat: upgrade to Angular v21.1.0#3311

Open
hoermillerj wants to merge 8 commits intoakveo:masterfrom
com-on-gmbh:feature/angular-v21-1
Open

feat: upgrade to Angular v21.1.0#3311
hoermillerj wants to merge 8 commits intoakveo:masterfrom
com-on-gmbh:feature/angular-v21-1

Conversation

@hoermillerj
Copy link
Contributor

📦 upgrades Angular and related dependencies to v21.1.0.
🐛 Removes NbTable providers since these are not needed.

Please read and mark the following check list before creating a pull request:

Short description of what this resolves:

Upgrades Angular and related dependencies to v21.1.0 and removes the deprecated _VIEW_REPEATER_STRATEGY that was removed from the latest @angular/cdk.

📦 upgrades  Angular and related dependencies to v21.1.0.
🐛 Removes NbTable providers since these are not needed.
♻️ fixes formatting
♻️ fixes formatting to match project preferences
♻️ fixes formatting to match project preferences
@hoermillerj hoermillerj marked this pull request as ready for review January 16, 2026 08:23
@hoermillerj hoermillerj changed the title Upgrade to Angular v21.1.0 feat: upgrade to Angular v21.1.0 Jan 16, 2026
@hoermillerj
Copy link
Contributor Author

@denStrigo Can you have a look? I don't know why the PR checks are failing. I ran npm ci locally and had no issues.

@hoermillerj hoermillerj mentioned this pull request Jan 16, 2026
📦 adds project version overrides to @angulöar/fire dependencies
@hoermillerj
Copy link
Contributor Author

@denStrigo I have added an override for @angular/compiler to @angular/fire, which resolves the npm ci issues. However the PR checks seem a bit flaky. I ran the packages build commands locally and they build fine.

♻️ fixes issues related to docs:build
🐛 added fixes in order to resolve pr build validation job errors
@soysergiogonza
Copy link

@hoermillerj Hi, you have news for this fix ?

@hoermillerj
Copy link
Contributor Author

hoermillerj commented Feb 10, 2026

@soysergiogonza sorry, no news so far. I recently added additional comits to the PR in order to fix some of the checks. I cannot reproduce the remaining failing pr checks. Everything works fine on my end and so far no one from the Akveo team has reacted to this pr. Last time I created the pr for the support for Angular v. 21.0.0 I had to remove changes I made to the unit tests, since it contained to many changes for a code review. Thats why I did not want to introduce to many code changes.

📦 sets the minimum required minor version of RxJS 7 to 7.5.5 to ensure that types are exported
👷 sets the installed RxJS Version in the build-packages-rxjs-7 PR check job to 7.5.5
@soysergiogonza
Copy link

Thanks

@hoermillerj
Copy link
Contributor Author

hoermillerj commented Feb 14, 2026

@denStrigo could you please have a look at the PR?

I updated the minimum required RxJS 7 minor version to 7.5.5 since this was the release where the types were added. This was causing the PR check build-packages-rxjs-7 to fail.

The unit tests would require a major rework due to changes in the change detection. They related to the Error: NG0100: ExpressionChangedAfterItHasBeenCheckedError.

The packages-smoke PR check would require me to update the packages in the packages-smoke project which uses @nguniversal/express-engine. This package was moved into the @angular/cli.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants