-
Notifications
You must be signed in to change notification settings - Fork 25
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
Upgrade to Angular 11 #252
Conversation
…cher into 237-updateangular
There is a build error, please check and fix it @MadLamprey. Thank you |
I have tested the changes and all current functionality is working as intended. LGTM! |
Version upgrades need to tested on multiple platforms similar to CATcher-org/CATcher#1242 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
System
OS: Windows 10 22H2
Browser: Chrome
Browser Version: Version 121.0.6167.185 (Official Build) (64-bit)
Local Angular Version: 11.2.14
Node Version: 14.21.2
npm version: 6.14.17
Actions Run
npm install
: success
npm run ng:serve:web
Tested Features
Switching between repositories
Filtering by labels
No bugs found
macOS works too :) SystemOS: macOS 13.3.1 (Ventura) Actions Runnpm install: success Tested FeaturesSwitching between repositories No bugs found |
SystemOS: Windows 11 23H2 Actions Runnpm install: success Tested FeaturesSwitching between repositories No bugs found |
SystemOS: Windows 11 23H2 Actions Runnpm install: success Tested FeaturesSwitching between repositories No bugs found |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary:
This pull request addresses some outdated packages in our application.
Fixes #237
Changes Made:
Proposed Commit Message: