-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
Add ability to alter foreign keys #299
Conversation
@karatakis Thanks for the PR. I left some comments) |
Thanks for the review @ikrivosheev. I will resolve them ASAP |
Thank you for the contribution. Just a question, isn't that we have |
@tyt2y3 |
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.
@karatakis thank you, some small comments.
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.
Would appreciate some tweaks
@karatakis thank you for contribution! @tyt2y3 LGTM! |
PR Info
Adds
Breaking Changes
prepare_foreign_key_create_statement_internal
of ForeignKeyBuilder to includeinside_table_single_alter: bool
Changes
prepare_foreign_key_drop_statement_internal
in ForeignKeyBuilder