Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

feat(pages): removed deprecated pages, added new pages, updated existing ones#2742

Merged
ovflowd merged 4 commits into
nodejs:mainfrom
ovflowd:feat/add-all-nodejs-org-pages
Sep 12, 2022
Merged

feat(pages): removed deprecated pages, added new pages, updated existing ones#2742
ovflowd merged 4 commits into
nodejs:mainfrom
ovflowd:feat/add-all-nodejs-org-pages

Conversation

@ovflowd

@ovflowd ovflowd commented Sep 10, 2022

Copy link
Copy Markdown
Member

Description

This PR syncs content from Nodejs.org by:

  • Removing all pages from Nodejs.dev that were removed from Nodejs.org
  • Adding all possible missing pages from Nodejs.org
    • Including Download Pages
    • Including Get Involved Pages
    • Including Learn Pages (Some were from Guides)
  • Updates the content of existing pages (Such as Download by Package Manager)

Finally this PR adjusts the following to make all this work:

  • Removes the deprecated "section" field from Markdown files
  • Updates the Sidebar Component to work with multiple section titles
    • Allow Sections to work with remote URLs
  • Updates Icons and Type definitions

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run npm run lint:js -- --fix and/or npm run lint:md -- --fix for my JavaScript and/or Markdown changes.
    • This is important as most of the cases your code changes might not be correctly linted
  • I have run npm run test to check if all tests are passing, and/or npm run test -- -u to update snapshots if I created and/or updated React Components.
  • I have checked that the build works locally and that npm run build work fine.
  • I've covered new added functionality with unit tests if necessary.

@ovflowd ovflowd added enhancement New feature or request docs labels Sep 10, 2022
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #2742 (d475d03) into main (fd57b87) will decrease coverage by 0.34%.
The diff coverage is 71.92%.

@@            Coverage Diff             @@
##             main    #2742      +/-   ##
==========================================
- Coverage   66.02%   65.68%   -0.35%     
==========================================
  Files         118      122       +4     
  Lines        1351     1390      +39     
  Branches      342      352      +10     
==========================================
+ Hits          892      913      +21     
- Misses        422      440      +18     
  Partials       37       37              
Impacted Files Coverage Δ
src/components/Header/index.tsx 100.00% <ø> (ø)
src/pages/download/releases.tsx 0.00% <0.00%> (ø)
src/pages/get-involved/code-learn.tsx 0.00% <ø> (ø)
src/pages/get-involved/collab-summit.tsx 0.00% <ø> (ø)
src/pages/get-involved/contribute.tsx 0.00% <0.00%> (ø)
src/pages/get-involved/index.tsx 0.00% <0.00%> (ø)
src/components/SearchBar/index.tsx 52.30% <40.00%> (-1.03%) ⬇️
src/hooks/useKeyPress.tsx 64.70% <64.70%> (ø)
src/components/TableOfContents/index.tsx 59.37% <83.33%> (+5.52%) ⬆️
src/components/Article/index.tsx 100.00% <100.00%> (ø)
... and 9 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label Sep 10, 2022
@github-actions github-actions Bot removed the create-preview Generate preview on staging.nodejs.dev label Sep 10, 2022
@github-actions

Copy link
Copy Markdown

Please find a preview at: https://staging.nodejs.dev/2742/

@benhalverson benhalverson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joesepi What do you think about replicating all the sub-pages for nodejs.org pages here?

For example, a code + learn hasn't happened since 2019 should this page still exist?

@ovflowd I see you renamed community to about. Why?

padding: 40px 30px 40px 0;

@media (max-width: 800px) {
@media (max-width: 900px) {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should have a sass variable for this value since it's used in multiple locations.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we should have a sass variable for this value since it's used in multiple locations.

Potential for next PR 😋

@ovflowd

ovflowd commented Sep 12, 2022

Copy link
Copy Markdown
Member Author

@ovflowd I see you renamed community to about. Why?

The Community page is removed from nodejs.org (@Trott gave some reasons why that page is deprecated as of now)

@ovflowd

ovflowd commented Sep 12, 2022

Copy link
Copy Markdown
Member Author

Also, @benhalverson regarding the Code + Learn if @Trott or @joesepi believe we can remove them, I'd be more than happy for doing so hehe

@ovflowd ovflowd force-pushed the feat/add-all-nodejs-org-pages branch from d475d03 to 9641029 Compare September 12, 2022 10:22
@ovflowd ovflowd force-pushed the feat/add-all-nodejs-org-pages branch from 0c994ba to c4c6f21 Compare September 12, 2022 10:24
@ovflowd ovflowd merged commit 0972d13 into nodejs:main Sep 12, 2022
@ovflowd ovflowd deleted the feat/add-all-nodejs-org-pages branch September 12, 2022 10:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

docs enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants