Skip to content

Ignore missing foreign key constraint in policies migration#23251

Merged
licitdev merged 3 commits intomainfrom
fix-23228-migration-foreign-key
Aug 8, 2024
Merged

Ignore missing foreign key constraint in policies migration#23251
licitdev merged 3 commits intomainfrom
fix-23228-migration-foreign-key

Conversation

@hanneskuettner
Copy link
Member

Scope

Gracefully handle the missing foreign key constraint in the up migration of policies. Normally the foreign key constraint should not be missing, but apparently it might. Since we're dropping the column in the next step anyways we should be safe to ignore the failed foreign key constraint.


Fixes #23228

@changeset-bot
Copy link

changeset-bot bot commented Aug 8, 2024

🦋 Changeset detected

Latest commit: 7b30ddd

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

@licitdev licitdev self-assigned this Aug 8, 2024
Copy link
Member

@licitdev licitdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@licitdev licitdev merged commit bb1b15a into main Aug 8, 2024
@licitdev licitdev deleted the fix-23228-migration-foreign-key branch August 8, 2024 10:16
@github-actions github-actions bot added this to the Next Release milestone Aug 8, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

directus_permissions_role_foreign missing when running migration from v10 -> v11

2 participants