Skip to content

build(deps): upgrade Angular 17 to Angular 18 with NgModule cleanup#111

Open
trungvose wants to merge 10 commits intomasterfrom
trung/v20-migration-part-5
Open

build(deps): upgrade Angular 17 to Angular 18 with NgModule cleanup#111
trungvose wants to merge 10 commits intomasterfrom
trung/v20-migration-part-5

Conversation

@trungvose
Copy link
Owner

Summary

  • Upgrade Angular core packages to v18.2.14 (from v17.3.12)
  • Upgrade @angular-builders/custom-webpack to 18.0.0
  • Upgrade @angular-eslint/* to 18.4.3
  • Upgrade @angular/cdk to 18.2.14
  • Upgrade ng-zorro-antd to 18.2.1 and @ant-design/icons-angular to 18.0.0
  • Upgrade ngx-quill to 26.x and Quill v1 → v2 (required for Angular 18 support)
  • Remove all remaining NgModules (AppRoutingModule, ProjectModule, ProjectRoutingModule, WorkInProgressModule, WorkInProgressRoutingModule, JiraControlModule)
  • Migrate to provideRouter() with standalone route configs (app.routes.ts, project.routes.ts, work-in-progress.routes.ts)
  • Clean up main.ts — remove BrowserModule, ReactiveFormsModule, NzSpinModule from importProvidersFrom
  • Remove deprecated extractCss build option from angular.json

Notes

  • TypeScript ~5.4.5 and zone.js ~0.14.10 were already compatible — no upgrade needed
  • Storybook v6→v7 migration deferred to a separate PR
  • Build system migration (esbuild) deferred — staying on webpack due to @angular-builders/custom-webpack dependency
  • Zoneless change detection and signal inputs are available but migration deferred
  • NzIconModule.forChild(NZ_JIRA_ICONS) preserved via route-level providers

Test plan

  • npm install runs without errors (no --force needed)
  • ng build (prod) compiles successfully
  • ng lint passes clean
  • App loads and core features work in browser
  • Rich text editor (Quill v2) works — create issue + edit issue description
  • Drag and drop works
  • Icons render correctly
  • Netlify preview deployment succeeds

🤖 Generated with Claude Code

@trungvose trungvose changed the base branch from trung/v20-migration-part-4 to master March 12, 2026 15:08
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.

1 participant