Skip to content
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

deprecate Primer::TimeAgoComponent in favor of Primer::RelativeTime #1692

Merged
merged 10 commits into from
Dec 8, 2022

Conversation

mxriverlynn
Copy link
Contributor

@mxriverlynn mxriverlynn commented Dec 8, 2022

Description

this PR deprecates Primer::TimeAgoComponent in favor of Primer::RelativeTime, and adds a migration guide for it

it also corrects an issue with all of the Lookbook links in the component documentation pages, where every link was missing the _preview part of the url for the component name.

screenshots

screenshot showing link to guide, on migrations page:
image

screenshot of guide, showing migration example:
image

screenshot of guide, showing relevant and changed arguments:
image

Integration

Does this change require any updates to code in production?

no immediate changes, though deprecations need to be cleaned up

Merge checklist

  • Added/updated tests
  • Added/updated documentation
  • Added/updated previews

@changeset-bot
Copy link

changeset-bot bot commented Dec 8, 2022

🦋 Changeset detected

Latest commit: 3e85ce9

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@primer/view-components Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@mxriverlynn mxriverlynn temporarily deployed to review-pr-1692 December 8, 2022 19:46 — with GitHub Actions Inactive
@github-actions github-actions bot added the ruby Pull requests that update Ruby code label Dec 8, 2022
@mxriverlynn mxriverlynn mentioned this pull request Dec 8, 2022
54 tasks
@mxriverlynn mxriverlynn temporarily deployed to github-pages December 8, 2022 19:50 — with GitHub Actions Inactive
@mxriverlynn mxriverlynn temporarily deployed to review-pr-1692 December 8, 2022 20:07 — with GitHub Actions Inactive
@mxriverlynn mxriverlynn temporarily deployed to github-pages December 8, 2022 20:11 — with GitHub Actions Inactive
@mxriverlynn mxriverlynn marked this pull request as ready for review December 8, 2022 20:13
@mxriverlynn mxriverlynn temporarily deployed to review-pr-1692 December 8, 2022 20:25 — with GitHub Actions Inactive
@mxriverlynn mxriverlynn temporarily deployed to github-pages December 8, 2022 20:30 — with GitHub Actions Inactive
@mxriverlynn mxriverlynn merged commit f50c77f into main Dec 8, 2022
@mxriverlynn mxriverlynn deleted the mxriverlynn/deprecate-time-ago branch December 8, 2022 21:43
@primer-css primer-css mentioned this pull request Dec 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ruby Pull requests that update Ruby code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants