-
-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Addon-docs: Add legacy transitional support for MDX1 #20747
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, these changes seem sensible.
So there's no way to transition partially from MDX1 -> 2? That seems OK if so.
@tmeasday unfortunately it's all or nothing for now. possible to enable piecemeal migration, but will require some API changes and i don't think it's worth the effort. this is really a stopgap solution and users should bite the bullet and do a full upgrade when they're ready. |
Issue: #20145
What I did
See attached migration notes:
Corresponding
mdx1-csf
PR here: https://github.com/storybookjs/mdx1-csf/pull/19/filesHow to test
yarn task --task sandbox --start-from auto --template cra/default-ts
Introduction.mdx
toIntroduction.stories.mdx
Examples/Introduction
storyChecklist
MIGRATION.MD
Maintainers
make sure to add the
ci:merged
orci:daily
GH label to it.["cleanup", "BREAKING CHANGE", "feature request", "bug", "documentation", "maintenance", "dependencies", "other"]