-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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 mysqli issue with Mautic 3 DB migrations #8961
Conversation
a67b402
to
84ee3ec
Compare
Codecov Report
@@ Coverage Diff @@
## staging #8961 +/- ##
==========================================
Coverage 35.25% 35.25%
Complexity 27889 27889
==========================================
Files 1731 1731
Lines 96725 96725
==========================================
Hits 34099 34099
Misses 62626 62626 |
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.
Confirmed the bug with the migration and the fix.
Is there a possibility to patch this back to Mautic 2.x as I am concerned that this bug will prevent some folk updating and hence migrating to Mautic 3.x.
@RCheesley No need to backport to 2.x as the affected migration doesn't exist in 2.x, only in 3. 😊 |
Ah OK - misunderstood the migration being from 2019 😊 |
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.
Thanks!
Please be sure you are submitting this against the staging branch.
Description:
This PR fixes Mautic 3 DB migration errors with the
mysqli
driver, which was reported in #8946.Steps to reproduce the bug:
Steps to test this PR:
gh pr checkout 8961
bin/console doctrine:migrations:migrate --no-interaction
to make sure your installation is up-to-date with DB migrations again.