Skip to content
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

Fix unique constraint / alter statement issues in marketplace migration #21721

Merged
merged 2 commits into from Mar 6, 2024

Conversation

paescuj
Copy link
Member

@paescuj paescuj commented Mar 6, 2024

Scope

What's changed:

  • Fix unique constraint / alter statement issues in marketplace migration by storing folder in new column directly, instead of renaming the name column
  • Fixed down script

Potential Risks / Drawbacks

None

Review Notes / Questions

  • Reproduced issue first
  • Tested & confirmed fix with:
    • SQLite
    • MariaDB
    • PostgreSQL

Extensions Before Migration (with all DB vendors)

Extensions · Directus

Extensions After Migration (with all DB vendors)

Note: I didn't migrate/re-add the deprecated "test:interface" extension

Extensions · Directus


Fixes #21710
Fixes #21714

Copy link

changeset-bot bot commented Mar 6, 2024

🦋 Changeset detected

Latest commit: 6d17b8f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@directus/api Patch
directus Patch

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

@rijkvanzanten rijkvanzanten self-assigned this Mar 6, 2024
@rijkvanzanten rijkvanzanten merged commit 1058ea1 into main Mar 6, 2024
4 checks passed
@rijkvanzanten rijkvanzanten deleted the fix-marketplace-migration-name branch March 6, 2024 17:32
@github-actions github-actions bot added this to the Next Release milestone Mar 6, 2024
fanmingfei pushed a commit to fanmingfei/directus that referenced this pull request Mar 7, 2024
…on (directus#21721)

* Fix unique constrain issues in marketplace migration

* Add changeset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
2 participants