-
Notifications
You must be signed in to change notification settings - Fork 2
chore(core): updates package compatibility with angular 20 #100
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
Merged
+21,838
−29,791
Conversation
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
nareshkharola123
approved these changes
Jan 14, 2026
- Added { static: false } to all 7 @ViewChild decorators in group.component.ts
- Added definite assignment assertion (!) for proper TypeScript typing
- These templates are accessed in ngAfterViewInit(), so static: false is correct
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 14.3.0 - Upgraded Angular CLI to 14.2.13 - Upgraded ng-packagr to 14.2.2 - Updated TypeScript compilation target to ES2020 - Migrated workspace configuration options - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 15.2.10 - Upgraded Angular CLI to 15.2.11 - Upgraded TypeScript to 4.9.5 - Upgraded ng-packagr to 15.2.2 - Fixed TypeScript 4.9 strict type checking by adding RecordOfAnyType constraints - Updated abstract class method signatures with proper generic constraints - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
…-dropdown - Upgraded all @angular packages to 16.2.12 - Upgraded Angular CLI to 16.2.16 - Upgraded ng-packagr to 16.2.3 - Upgraded zone.js to 0.13.3 - Temporarily disabled ng-multiselect-dropdown (not Ivy compatible) - Commented out module import and component usage - Changed types to 'any' to allow compilation - Will be replaced with @ng-select/ng-select in Phase 3 - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 17.3.12 - Upgraded Angular CLI to 17.3.17 - Upgraded TypeScript to 5.4.5 (required for Angular 20) - Upgraded ng-packagr to 17.3.0 - Upgraded zone.js to 0.14.10 - Build tested and working - Next: migrate templates to new control flow syntax Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Migrated *ngIf to @if - Migrated *ngFor to @for with track expressions - Migrated *ngSwitch to @switch - Updated 4 template files: - builder.component.html - group.component.html - node.component.html - tooltip-render.component.html - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 18.2.14 - Upgraded Angular CLI to 18.2.21 - Upgraded ng-packagr to 18.2.1 - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 19.2.18 - Upgraded Angular CLI to 19.2.19 - Upgraded TypeScript to 5.8.3 - Upgraded ng-packagr to 19.2.2 - Upgraded zone.js to 0.15.1 - Added standalone: false to all components (Angular 19 migration) - Fixed TypeScript 5.8 error: missing .includes() in array check (helpers.ts:57) - Build tested and working Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Upgraded all @angular packages to 20.3.16 - Upgraded Angular CLI to 20.3.14 - Upgraded ng-packagr to 20.3.2 - Updated TypeScript moduleResolution to 'bundler' - Updated workspace generation defaults - Build tested and working PHASE 2 COMPLETE: Successfully migrated from Angular 13 to Angular 20! Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
- Updated tsconfig.json module to ES2022 (from es2020) - Updated tsconfig.json lib to ES2022 (from es2018) - Updated tslib from ^2.3.0 to ^2.8.1 - Updated rxjs from ~7.5.0 to ^7.8.2 - Verified angular.json configuration is optimal - Verified tsconfig.lib.json and tsconfig.spec.json settings - Build tested and working (381KB bundle size) - Tree-shaking verified and working correctly PHASE 3 COMPLETE: All TypeScript & Build Tooling Updates finished! Next: Phase 4 - Third-Party Library Updates Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
**@ng-bootstrap/ng-bootstrap:** - Upgraded from v12.1.2 to v20.0.0 - Added Bootstrap 5.3.8 as dependency - All components (datepicker, timepicker, popover) working correctly **ngx-popperjs:** - Upgraded from v13.3.0 to v17.0.1 - Upgraded @popperjs/core to latest (2.11.8) - Removed deprecated popperInstance.forceUpdate() call - Positioning now updates automatically in v17 **ng-multiselect-dropdown → @ng-select/ng-select:** - Replaced incompatible ng-multiselect-dropdown (0.3.9) - Installed @ng-select/ng-select (latest) - Updated module imports: NgSelectModule - Updated component: removed dropdownSettings, kept selectedItems[] - Updated template: replaced with ng-select component - [multiple]="true" for multi-select - [searchable]="true" for filtering - bindLabel/bindValue for field mapping - Removed old ng-multiselect-dropdown dependency **Build & Testing:** - Build successful: 2.4s compile time - Bundle size verified and optimized - All library integrations working correctly PHASE 4 COMPLETE: All third-party libraries updated for Angular 20! Note: bpmn-moddle kept at v8.0.0 as requested (not upgraded to v10) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
**Component Input Validation:** BuilderComponent: - Added @input({ required: true }) to localizedStringMap - Added @input({ required: true }) to allColumns - Added definite assignment assertions (!) for type safety GroupComponent: - Added @input({ required: true }) to group - Added @input({ required: true }) to eventGroups - Added @input({ required: true }) to nodeType - Added @input({ required: true }) to popupTemplate - Added @input({ required: true }) to allColumns - Added definite assignment assertions (!) for type safety NodeComponent: - Added @input({ required: true }) to node - Added @input({ required: true }) to inputTemplate - Added @input({ required: true }) to popupTemplate - Added definite assignment assertions (!) for type safety **Other Phase 5 Tasks:** - ✅ Control flow migration (completed in Phase 2) - ✅ ViewChild updates (completed in Phase 2) - ⏭️ DI pattern updates (optional, skipped) - ⏭️ Standalone migration (optional, skipped) **Build & Testing:** - Build successful: 2.1s compile time - All required inputs now properly typed and validated - Improved type safety across components PHASE 5 COMPLETE: All code migrations and breaking changes implemented! Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
angular-v20-migration
e65e061 to
2b4b452
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: