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
Consolidate content versioning #22227
Conversation
🦋 Changeset detectedLatest commit: d50f17a The changes in this PR will be included in the next version bump. This PR includes changesets to release 21 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
So far working well in my tests versions get properly migrated and seem to function as expected/before in my tests.
Just this one question about the fallback but thats not a blocker as it should already be an edge case to get there.
Scope
What's changed:
delta
field to thedirectus_versions
tablePotential Risks / Drawbacks
Review Notes / Questions
versions
field in thedirectus_revisions
table will be removed in a subsequent releasedelta
field indirectus_versions
is empty, the existing behaviour of fetching fromdirectus_revisions
remainsRelated to #17894