Skip to content
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.

Sign up for GitHub

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

Jump to bottom

feature: update angular #4454

Draft
wants to merge 9 commits into
base: develop
Choose a base branch
from
Draft

feature: update angular #4454

wants to merge 9 commits into from

Conversation

itsmostafa
Copy link
Collaborator

@itsmostafa itsmostafa commented Feb 21, 2025 •
edited
Loading

  • Bring ngx-ellispis in house as its not being maintained and the code is lean enough to be supported directly.
  • Remove duplicate load of zone.js
  • Update all packages to versions that support Angular 19
  • Update ALL components to include standalone: false flag
  • Refactor sass styling files to remove all deprecation warnings
  • fix minor bugs
  • Update to Angular 19

🗣 Description

💭 Motivation and context

🧪 Testing

successfully tested locally

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • All new and existing tests pass.

@itsmostafa itsmostafa self-assigned this Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@itsmostafa itsmostafa

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@itsmostafa