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

[13.0][IMP] apriori: edi_oca -> edi #4452

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

MiquelRForgeFlow
Copy link
Contributor

@MiquelRForgeFlow MiquelRForgeFlow commented Jun 5, 2024

Was done in OCA/edi#877.

It was commented in #4384:
Selection_3378

@pedrobaeza pedrobaeza added this to the 13.0 milestone Jun 5, 2024
@pedrobaeza
Copy link
Member

I don't know the history about this, so others are more suitable to judge this move. I see it weird, as AFAIK everything was moved to _oca suffix.

@MiquelRForgeFlow
Copy link
Contributor Author

MiquelRForgeFlow commented Jun 5, 2024

Yeah, in v14 was done the rename from edi to edi_oca. But later, when we made the backport to v12 we used v14 as reference, and we didn't change the module name.

@pedrobaeza
Copy link
Member

Please @hbrunn can you confirm this is OK?

Copy link
Member

@hbrunn hbrunn left a comment

Choose a reason for hiding this comment

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

I think we need this indeed for migrating to v13, and then we'll have to rename it back for migrating to v14:

https://github.com/OCA/edi/tree/12.0/edi_oca
https://github.com/OCA/edi/tree/13.0/edi
https://github.com/OCA/edi/tree/14.0/edi_oca

@pedrobaeza pedrobaeza merged commit 00aecbd into OCA:13.0 Jun 10, 2024
3 checks passed
@pedrobaeza pedrobaeza deleted the 13.0-imp-edi_oca-migration branch June 10, 2024 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants