Skip to content
This repository was archived by the owner on Aug 18, 2025. It is now read-only.

Handle versioned links in relative-to-root-links#555

Merged
kylegach merged 1 commit into
mainfrom
versioned-docs-links
Jun 7, 2023
Merged

Handle versioned links in relative-to-root-links#555
kylegach merged 1 commit into
mainfrom
versioned-docs-links

Conversation

@kylegach

@kylegach kylegach commented Jun 7, 2023

Copy link
Copy Markdown
Collaborator

This approach will eventually1 replace LinkWithVersion. It allows for functional links when viewing on GitHub.

  • ../../../release-6-5/docs/api/csf.md/docs/6.5/react/api/csf.md
  • Add tests

Footnotes

  1. For now, LinkWithVersion remains available in MDX, so that I can update the actual docs content in the monorepo. After that, I'll remove that component, as it is no longer necessary.

- ../../../release-6-5/docs/api/csf.md -> /docs/6.5/react/api/csf.md
- Add tests
@kylegach kylegach requested a review from winkerVSbecks June 7, 2023 15:50
@netlify

netlify Bot commented Jun 7, 2023

Copy link
Copy Markdown

Deploy Preview for storybook-frontpage ready!

Name Link
🔨 Latest commit a43556f
🔍 Latest deploy log https://app.netlify.com/sites/storybook-frontpage/deploys/6480a749c06c7d00088b0ae9
😎 Deploy Preview https://deploy-preview-555--storybook-frontpage.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@kylegach kylegach requested a review from jonniebigodes June 7, 2023 15:50

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for adding tests 🙏🏽

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the ultra-quick review!

@kylegach kylegach merged commit 7720e17 into main Jun 7, 2023
@kylegach kylegach deleted the versioned-docs-links branch June 7, 2023 16:12
kylegach added a commit to storybookjs/storybook that referenced this pull request Jun 7, 2023
kylegach added a commit that referenced this pull request Jun 8, 2023
- No longer used, per #555 & storybookjs/storybook#22966
- Simplify `relativeToRootLinks`
@kylegach kylegach mentioned this pull request Jun 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants