documentation icon indicating copy to clipboard operation
documentation copied to clipboard

add page Database migrations

Open petersg83 opened this issue 2 years ago • 4 comments

What does it do?

It adds the page Database migrations

Why is it needed?

It is needed to document a feature that is used in the migration guide for 4.1.10

Related issue(s)/PR(s)

Migration guide : https://github.com/strapi/documentation/pull/885

petersg83 avatar May 10 '22 12:05 petersg83

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
documentation ✅ Ready (Inspect) Visit Preview Oct 6, 2022 at 9:43AM (UTC)

vercel[bot] avatar May 10 '22 12:05 vercel[bot]

Thanks again @petersg83 ! I had a second look at this documentation and improved the styling and wording to be more consistent with the rest of the documentation. What do you think?

It looks almost fine to me, but I have one question/comment: is the down() function really not usable at the moment? We don't feel comfortable documenting something that actually does not work. 😅

pwizla avatar May 19 '22 12:05 pwizla

Thanks again @petersg83 ! I had a second look at this documentation and improved the styling and wording to be more consistent with the rest of the documentation. What do you think?

It looks almost fine to me, but I have one question/comment: is the down() function really not usable at the moment? We don't feel comfortable documenting something that actually does not work. sweat_smile

We documented the state option for lifecycles before they even worked. They still dont but will in the next release :P

I think it's fine as long as we say "this is coming soon" or something. Else we can just comment out that line and hide it til it's ready.

derrickmehaffy avatar May 20 '22 20:05 derrickmehaffy

This pull request has been mentioned on Strapi Community Forum. There might be relevant details there:

https://forum.strapi.io/t/database-migration-deployment-questions/434/41

strapi-bot avatar May 31 '22 21:05 strapi-bot

I think @pwizla is right, regarding not showing the down() function, I don't think it's coming anytime soon

petersg83 avatar Oct 06 '22 08:10 petersg83

done :)

petersg83 avatar Oct 06 '22 09:10 petersg83