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
Update previous-releases.mdx #6676
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
d60a4de
to
b48e70c
Compare
…ases Signed-off-by: Steven R. Loomis <srl295@gmail.com>
b48e70c
to
f82530d
Compare
Related issue #6669 |
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.
Mostly LGTM.
Except, if we rename it to "Node.js Releases", we should probably also rename the page? From /about/previous-releases
to /about/releases
(or similar).
If we rename the link we break a lot of pages. Im fine renaming it although we need a redirect ;) |
I'm not sure how to redirect best. Could one of you update the pr? Or have that as a follow on? This already improved things. |
IMHO package-manager, prebuilt-binaries and source-code can receive mention of "previous releases" |
sounds like it should become a component? |
I didn't get what you want to do. |
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.
blocking this explicitly until a redirect is in place - i can guarantee it will break stuff without
How does this break? It doesn't rename pages. |
The renaming can be done on a follow up PR. |
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!
The renaming of the file did not happen yet
Lighthouse Results
|
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 !
Description
per discussion
Validation
verify titles are changed
Related Issues
Check List
npm run format
to ensure the code follows the style guide.npm run test
to check if all tests are passing.npx turbo build
to check if the website builds without errors.