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

[12.0][MIG][ADD] openupgrade migrations #99

Open
wants to merge 8 commits into
base: 12.0
Choose a base branch
from
Open

Conversation

vvrossem
Copy link
Contributor

@vvrossem vvrossem commented Jun 2, 2020

THIS PR IS NOT TO BE MERGED

The 12.0-openupgrade branch is used for OpenUpgrade migrations.

easy_my_coop

This solves the critical error duplicate key value violates unique constraint "res_groups_name_uniq" raised when using openupgrade.
This error is due to the fact that group_energiris_user has been renamed as group_easy_my_coop_user and group_energiris_manager in group_easy_my_coop_manager

website_recaptcha_reloaded

9.0 -> 12.0

@vvrossem vvrossem requested a review from remytms June 2, 2020 16:22
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2020

Codecov Report

Merging #99 (805886d) into 12.0 (5fd6aa1) will decrease coverage by 3.90%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             12.0      #99      +/-   ##
==========================================
- Coverage   59.40%   55.49%   -3.91%     
==========================================
  Files         119      100      -19     
  Lines        4069     3546     -523     
  Branches      618      564      -54     
==========================================
- Hits         2417     1968     -449     
+ Misses       1549     1500      -49     
+ Partials      103       78      -25     
Impacted Files Coverage Δ
easy_my_coop/models/subscription_request.py 55.55% <0.00%> (-0.66%) ⬇️
easy_my_coop_connector/wizards/__init__.py
..._coop_connector/components/abstract_emc_adapter.py
easy_my_coop_connector/models/__init__.py
...op_connector/components/account_payment_adapter.py
easy_my_coop_connector/components/__init__.py
easy_my_coop_connector/components/emc_backend.py
easy_my_coop_connector/models/account_account.py
easy_my_coop_connector/models/account_journal.py
...nnector/components/subscription_request_adapter.py
... and 10 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@vvrossem vvrossem changed the title [12.0][MIG][ADD] easy_my_coop: openupgrade.rename_xmlids() [12.0][MIG][ADD] openupgrade migrations Jun 3, 2020
@houssine78
Copy link
Contributor

@vvrossem I think that it has been used for two or three migration. Can we go for merging it ?

@vvrossem
Copy link
Contributor Author

vvrossem commented Oct 6, 2020

@remytms ok to merge ?

@houssine78
Copy link
Contributor

@remytms can we merge this ?

@remytms
Copy link
Member

remytms commented Mar 29, 2021

We should no merge this PR as it will force to install openupgradelib on production servers.

So do no merge this PR. It stands here to not remove the 12.0-openugrade branch.

carmenbianca and others added 5 commits May 13, 2022 14:16
…igration

Previously, for some reason, these did not default to True, or somehow
lost their truthiness during migration from v9 to v12. Whatever the
case, this should make sure that doesn't happen.

Signed-off-by: Carmen Bianca Bakker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants