-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
docs: clarify MDX version used #5575
Conversation
✔️ [V2] 🔨 Explore the source changes: 58c78ac 🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/61435cef385cb20007a9137f 😎 Browse the preview: https://deploy-preview-5575--docusaurus-2.netlify.app |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-5575--docusaurus-2.netlify.app/ |
Size Change: -4 B (0%) Total Size: 826 kB ℹ️ View Unchanged
|
The link still directs to v2. Shouldn't the v1 link be used? |
Does MDX have site designed for the current v1? https://v1.mdxjs.com/ is very outdated site. |
Hmm, this link you posted in the other issue seems good for now? https://mdx-git-renovate-babel-monorepo-mdx.vercel.app/ |
This is actually preview deploy from one of PR from mdxjs repo. I'm not sure if it should be used for this link, but it might be worth using for MDX playground link on 26 line. |
I see, interesting that the deploy preview link is so human readable🧐 |
Motivation
See #4029 (comment)
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)