-
-
Notifications
You must be signed in to change notification settings - Fork 10.6k
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
Data & Fixture Migrations v005 #6628
Conversation
9757294
to
4a8cf11
Compare
@sebgie in the last migration here, I've added a new |
Yes, the types are meant to reflect the specification. Tbf,
|
Fair point, I was just following along from what was there, I don't think there's a particular reason to keep them so short -
|
eef614a
to
9ee52ce
Compare
refs TryGhost#6301 - bump the default version & update corresponding test - add empty task folders for 005 data & fixture migrations - update tests to cover the new 005 upgrades
refs TryGhost#6301 - move the temporary `fixClientSecret` function from migration.init into being a proper fixture migration task - update the tests accordingly
refs TryGhost#6301 - column is not used and we'll be adding a visibility column to serve the intended purpose
refs TryGhost#6301, TryGhost#6165 - visibility is added as a new column on posts, tags and users. - has a relevant default value for each table
refs TryGhost#6301, TryGhost#6255 - new, extra-long, column for storing mobiledoc content format
refs TryGhost#6301, TryGhost#6534 - adds facebook and twitter columns, which should contain urls
refs TryGhost#6301, TryGhost#6399 - new scheduler client will be used by any web app that handles time and calls back to the scheduling API at the right time - new scheduler client will need to be confidential, rather than public, hence the 'web' type instead of 'ua' - adds validation to client type that it must have a type of 'ua', 'web', or 'native'
refs TryGhost#6301, TryGhost#4176 Add migration for: - 5 new client permissions - 15 relations between the admin, editor & author role and the 5 new permissions - updates to tests to show that permissions get updated properly
This is a brand new PR for pushing migrations that need to happen in 005 📚 🚀
Back in #6302, we had migrations that were auto-calculated from the schema / fixtures files. This was easier to do at face value, but felt risky because it was impossible to test and verify that changes would work correctly. The new version requires us to write more code, but gives us more guarantee that the upgrades will work now and in the future.
From here on, each change to the schema or fixtures needs to follow some rules: