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

feat: 🎸 Added governance page#1336

Merged
benhalverson merged 5 commits into
nodejs:mainfrom
benhalverson:governance-page
Jun 16, 2021
Merged

feat: 🎸 Added governance page#1336
benhalverson merged 5 commits into
nodejs:mainfrom
benhalverson:governance-page

Conversation

@benhalverson

Copy link
Copy Markdown
Member

Description

Adding /governance page
The existing page for nodejs.org is https://nodejs.org/en/about/governance/

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label May 18, 2021
@github-actions github-actions Bot removed the create-preview Generate preview on staging.nodejs.dev label May 18, 2021
@github-actions

Copy link
Copy Markdown

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

@benhalverson

Copy link
Copy Markdown
Member Author

Comment thread src/pages/governance.tsx Outdated
Comment thread test/pages/governance.test.tsx Outdated
@codecov-commenter

codecov-commenter commented May 19, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1336 (c425558) into main (99362e8) will increase coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1336      +/-   ##
==========================================
+ Coverage   86.22%   86.32%   +0.10%     
==========================================
  Files          68       69       +1     
  Lines         762      768       +6     
  Branches      218      218              
==========================================
+ Hits          657      663       +6     
  Misses        105      105              
Impacted Files Coverage Δ
src/pages/governance.tsx 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 99362e8...c425558. Read the comment docs.

Comment thread test/__fixtures__/page.tsx Outdated
},
});

export const createGovernanceData = (): Page => ({

@manishprivet manishprivet May 20, 2021

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.

I've noticed that apart from this, #1345, #1336, #1298, #1297 and #1296 use the exact same data for testing but use different function name. And I think it'll be created again for any new static page that we're going to add in issues like #1341.

I think we can keep a generic function for this type of PRs. Maybe we can change the function name to a generic one here, and then use it as an example for all other old / upcoming PRs.

@joesepi

joesepi commented May 21, 2021

Copy link
Copy Markdown
Member

Needs left sidebar. Thanks!!

@benhalverson benhalverson added the create-preview Generate preview on staging.nodejs.dev label Jun 16, 2021
@github-actions github-actions Bot removed the create-preview Generate preview on staging.nodejs.dev label Jun 16, 2021
@github-actions

Copy link
Copy Markdown

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

@benhalverson benhalverson requested a review from joesepi June 16, 2021 08:41

@joesepi joesepi 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.

LGTM. Thanks!

@benhalverson benhalverson merged commit 39e04d3 into nodejs:main Jun 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants