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

[Adjustment] Update new migrations not to execute if the changes already exist in db #12364

Merged
merged 1 commit into from
Feb 17, 2021

Conversation

GSadee
Copy link
Member

@GSadee GSadee commented Feb 17, 2021

Q A
Branch? 1.9
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets
License MIT

I'm adding these if statements not to execute migrations if the changes already exist in the database. It could resolve potential problems for upgrading Sylius from 1.8 to 1.9 with installed the newest RefundPlugin.

@GSadee GSadee added DX Issues and PRs aimed at improving Developer eXperience. Maintenance CI configurations, READMEs, releases, etc. labels Feb 17, 2021
@GSadee GSadee requested a review from a team as a code owner February 17, 2021 14:02
@lchrusciel
Copy link
Member

I'm only wondering if we should cover the same case for down method

@lchrusciel lchrusciel merged commit f62de2a into Sylius:1.9 Feb 17, 2021
@lchrusciel
Copy link
Member

Thanks, Grzegorz! 🎉

@GSadee GSadee deleted the not-execute-migrations branch February 17, 2021 14:12
GSadee added a commit that referenced this pull request Feb 17, 2021
…o execute if the changes already exist in db (GSadee)

This PR was merged into the 1.9 branch.

Discussion
----------

| Q               | A
| --------------- | -----
| Branch?         | 1.9
| Bug fix?        | no
| New feature?    | no
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | fixes #12364 (comment)
| License         | MIT


Commits
-------

41fe59e [Adjustment] Update down methods of new migrations not to execute if the changes already exist in db
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DX Issues and PRs aimed at improving Developer eXperience. Maintenance CI configurations, READMEs, releases, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants