Skip to content

Conversation

@ovflowd
Copy link
Member

@ovflowd ovflowd commented Jul 10, 2023

Description

This PR introduces a simple Time component that fulfils the specification presented on the issue.

It also updates the en locale to use en-GB

Related Issues

Fixes #5481

@ovflowd ovflowd requested a review from a team as a code owner July 10, 2023 15:22
@vercel
Copy link

vercel bot commented Jul 10, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nodejs-org ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2023 2:50am
nodejs-org-stories ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2023 2:50am

@vercel vercel bot temporarily deployed to Preview – nodejs-org July 10, 2023 15:22 Inactive
@ovflowd
Copy link
Member Author

ovflowd commented Jul 10, 2023

@mikeesto I've made a clean PR (just because I accidentally deleted the branch on upstream)

(Actually, this branch should have been done on the fork to begin with... but hey, Codespaces default remote was set to upstream)

@ovflowd ovflowd changed the title chore: updated english to use en-GB locale feat: introduced generic time component Jul 10, 2023
@vercel vercel bot temporarily deployed to Preview – nodejs-org-stories July 10, 2023 15:23 Inactive
@github-actions
Copy link
Contributor

github-actions bot commented Jul 10, 2023

📦 Next.js Bundle Analysis for nodejs.org

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@vercel vercel bot temporarily deployed to Preview – nodejs-org-stories July 10, 2023 15:26 Inactive
@vercel vercel bot temporarily deployed to Preview – nodejs-org July 10, 2023 15:27 Inactive
@vercel vercel bot temporarily deployed to Preview – nodejs-org-stories July 11, 2023 02:49 Inactive
@vercel vercel bot temporarily deployed to Preview – nodejs-org July 11, 2023 02:50 Inactive
Copy link
Member

@mikeesto mikeesto left a comment

Choose a reason for hiding this comment

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

Not sure why there is a diff for package-lock? Otherwise LGTM.

Copy link
Contributor

@bmuenzenmeyer bmuenzenmeyer left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@shanpriyan shanpriyan left a comment

Choose a reason for hiding this comment

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

LGTM!

@ovflowd
Copy link
Member Author

ovflowd commented Jul 11, 2023

Not sure why there is a diff for package-lock? Otherwise LGTM.

A little audit fix

@ovflowd ovflowd merged commit b483452 into main Jul 11, 2023
@ovflowd ovflowd deleted the feat/generic-time-component branch July 11, 2023 09:13
umairraza96 pushed a commit to umairraza96/nodejs.org that referenced this pull request Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create <Time/> component

6 participants