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

[16.0][MIG] base_address_extended #3769

Merged
merged 1 commit into from
May 20, 2023

Conversation

marielejeune
Copy link
Contributor

@marielejeune marielejeune commented Mar 14, 2023

It seems that base_address_city was merged into base_address_extended. Is there something else to do than renaming the xmlids?
Edit: the modules will be merged with pre-migration script of base, as they are listed in apriori.py

@marielejeune
Copy link
Contributor Author

#3681

@sbidoul sbidoul added this to the 16.0 milestone Mar 14, 2023
@sbidoul
Copy link
Member

sbidoul commented Mar 15, 2023

/ocabot migration base_address_extended

Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't have to rename anything, as it's already handled in the call of update_module_names openupgradelib method in base module.

@marielejeune marielejeune force-pushed the 16.0-base_address_extended-mle branch from 1c0c108 to ebbd677 Compare March 27, 2023 13:58
@marielejeune marielejeune force-pushed the 16.0-base_address_extended-mle branch from ebbd677 to 10b8463 Compare March 27, 2023 14:43
@@ -70,7 +70,7 @@ Module coverage 15.0 -> 16.0
+-------------------------------------------------+----------------------+-------------------------------------------------+
| |del| base_address_city | | |
Copy link
Member

@pedrobaeza pedrobaeza Mar 28, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| |del| base_address_city | | |
| |del| base_address_city | Done | Merged into base_address_extended |

As this has been already tackled in https://github.com/OCA/OpenUpgrade/blob/2d7f8de7289fdf36e0ac48f41ee52b6138190399/openupgrade_scripts/apriori.py#LL34C1-L34C1

@pedrobaeza pedrobaeza merged commit 2d7f8de into OCA:16.0 May 20, 2023
@sbidoul sbidoul deleted the 16.0-base_address_extended-mle branch May 20, 2023 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants