Angular 12 #1242
Merged
Angular 12 #1242
Conversation
This file contains 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
The cosmiconfig-typescript-loader library used by graphQL-codegen requires ts-node v10 as peer-dependency. Since no other libraries depend on ts-node, this update should be safe to perform.
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #1242 +/- ##
==========================================
+ Coverage 53.13% 54.77% +1.63%
==========================================
Files 105 105
Lines 2951 2859 -92
Branches 549 503 -46
==========================================
- Hits 1568 1566 -2
Misses 1030 1030
+ Partials 353 263 -90 ☔ View full report in Codecov by Sentry. |
The latest major version (v2.16.5) uses cosmiconfig-typescript-loader@^4.3.0 instead of v4.3.0. This causes the npm installer to pull the latest major version of cosmiconfig-typescript-loader@4.4.0, which requires Node v14.21.3
|
Kindly provide your |
SystemOS: macOS Ventura 13.6.3 Actions Run
Tested Phases
No bugs found |
SystemOS: Windows 11 22H2 Actions Run
Tested Phases
No bugs found |
SystemOS: ubuntu 22.04 Actions Run
Tested Phases
No bugs found |
gycgabriel
approved these changes
Feb 19, 2024
vigneshsankariyer1234567890
added a commit
that referenced
this pull request
Mar 4, 2024
* Fix broken duplicate link (#1233) Fix the broken link of a duplicate issue Currently, the user cannot open the link to a duplicate issue when opening an issue, as described in #1228. The links now work as expected. * Add whitespace validation (#1237) * Add whitespace validation * Update whitespace validation for new issue * Update whitespace validation for title of new issues * Update whitespace validation for title of new issues * Move validators into core * Update import order --------- Co-authored-by: Misra Aditya <e1096355@u.nus.edu> * Fix uncaught errors when attempting to access an invalid route There is an uncaught error when the users click on an invalid internal link in Markdown or enter an invalid link in browser. Internal links are unlikely to be used for bug reporting and are more likely to be invalid. Let's show an error toaster and stop the navigation when clicking on an internal link in Markdown. Also, redirect the users to the login page if the users enter invalid link in browser. * Set default branch to `main` Previously, image uploads depend on the user's default branch. Now, we set the branch for image upload to be `main`. Images will be uploaded to `main` as a result. --------- Co-authored-by: Chee Hong <c.h.wong2606@gmail.com> * Preserve linebreaks (#1241) With preserving linebreaks, subset list items are rendered as a list, and paragraph rendering is the same as Github. * Faulty list view when back navigating (#1243) Issue table settings such as page index are not saved when table is re-mounted. This behavior inconveniences users as their settings are reset everytime they navigate to a specific issue and back. Let's lift up the table settings of each mounted table to a service which the tables pull from when mounted. * Upgrade to Angular 12 (#1242) Some of our packages are old and outdated. We should actively maintain and keep these packages up-to-date so it is easier to maintain in the future. Let's upgrade to Angular 12 to keep our packages up-to-date. * Fix markdown blockquote preview difference (#1245) Due to DOMPurify, the content used for preview is different. However, given that ngx-markdown already has sufficient sanitation by default, we remove sanitation by DOMPurify. * Create release for v3.5.3 --------- Co-authored-by: Nguyen <87511888+nknguyenhc@users.noreply.github.com> Co-authored-by: AdityaMisra <114080910+MadLamprey@users.noreply.github.com> Co-authored-by: Misra Aditya <e1096355@u.nus.edu> Co-authored-by: NereusWB922 <107099783+NereusWB922@users.noreply.github.com> Co-authored-by: Chee Hong <c.h.wong2606@gmail.com> Co-authored-by: Arif Khalid <88131400+Arif-Khalid@users.noreply.github.com>
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.
Summary:
This pull request addresses some of the outdated packages in our application.
Changes Made:
Upgrade to Angular 12 using
ng updateUpdate dependency
ngx-markdown@12.0.1(from@11.2.0) to support angular 12Update dependency
ts-node@^10.9.2from@^7.0.1Freeze dependency
@graphql-codegen/cli@2.16.4from^2.6.42.16.5, which setscosmiconfig-typescript-loader@^4.3.0as dependency. The latest major version (4.4.0) ofcosmiconfig-typescript-loaderonly supportsNode v14.21.3and aboveProposed Commit Message: