-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
docs: add data migration scripts section #12975
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: add data migration scripts section #12975
Conversation
|
Someone is attempting to deploy a commit to the medusajs Team on Vercel. A member of the Team first needs to authorize it. |
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.
Thank you for your contribution! I've added comments for fixes and changes.
www/apps/book/app/learn/fundamentals/data-models/write-migration/page.mdx
Outdated
Show resolved
Hide resolved
www/apps/book/app/learn/fundamentals/data-models/write-migration/page.mdx
Outdated
Show resolved
Hide resolved
www/apps/book/app/learn/fundamentals/data-models/write-migration/page.mdx
Outdated
Show resolved
Hide resolved
www/apps/book/app/learn/fundamentals/data-models/write-migration/page.mdx
Outdated
Show resolved
Hide resolved
Co-authored-by: Shahed Nasser <[email protected]>
should be all done |
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.
LGTM thanks!
Thank you for the edits really, now it is much more understandable especially for new users. Always appreciate your work, the medusa docs are some of the best I've seen 👍 |
I finally had some free time to add this section on data migration scripts, closes #12738
I copied most of the description from the release notes from v2.3.0, as I found them to be well written, and I simply added a basic example to illustrate the function signature(taken from packages/medusa/src/migration-scripts/migrate-product-shipping-profile.ts).
Let me know if I should change anything. I made sure to run it locally to check that it looked ok(which took me a few minutes, you are supposed to have a .env in
www/apps/book
otherwise the build fails. Very simple and straightforward but it isn't mentioned in the docs contribution guide, should it be added?)