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.

Sign up for GitHub

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

Jump to bottom

Add Migrator methods to create DB foreign keys #1019

Merged
merged 19 commits into from
Jul 19, 2022
Merged

Add Migrator methods to create DB foreign keys #1019

merged 19 commits into from
Jul 19, 2022

Conversation

mvorisek
Copy link
Member

@mvorisek mvorisek commented Jul 7, 2022 •
edited
Loading

tested with all tests, fix various related integrity issues

FK can be setup from test like:

$this->createMigrator()->createForeignKey($join);
$this->createMigrator()->createForeignKey($hasOneOrHasManyReference);

@mvorisek mvorisek force-pushed the ref_integrity branch 30 times, most recently from 19d282c to 1c57657 Compare July 11, 2022 23:43
@mvorisek mvorisek added the MAJOR label Jul 18, 2022
@mvorisek mvorisek changed the title Add Migrator methods to setup FK Add Migrator methods to setup foreign keys Jul 18, 2022
@mvorisek mvorisek changed the title Add Migrator methods to setup foreign keys Add Migrator methods to setup DB foreign keys Jul 18, 2022
@mvorisek mvorisek changed the title Add Migrator methods to setup DB foreign keys Add Migrator methods to create DB foreign keys Jul 18, 2022
@mvorisek mvorisek force-pushed the ref_integrity branch 2 times, most recently from 62cd837 to 81713d3 Compare July 18, 2022 23:27
@mvorisek mvorisek marked this pull request as ready for review July 18, 2022 23:35
@mvorisek mvorisek merged commit 8d738d8 into develop Jul 19, 2022
@mvorisek mvorisek deleted the ref_integrity branch July 19, 2022 08:01
@mvorisek mvorisek mentioned this pull request Dec 20, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@georgehristov georgehristov Awaiting requested review from georgehristov

@DarkSide666 DarkSide666 Awaiting requested review from DarkSide666

@abbadon1334 abbadon1334 Awaiting requested review from abbadon1334

@mkrecek234 mkrecek234 Awaiting requested review from mkrecek234

Assignees
No one assigned
Labels
MAJOR
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mvorisek