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

Ignore manifest check for l10n_generic_coa in 17.0 and above #609

Closed
wants to merge 3 commits into from

Conversation

obayit
Copy link
Contributor

@obayit obayit commented May 3, 2024

Since Odoo 17.0 they removed the module l10n_generic_coa, but they kept its directory and with only the init.py file.

This causes manifest check to fail if strict was True.

This commit causes addons_config to ignore this module for 17.0 and above.

obayit added 3 commits May 3, 2024 15:24
Since Odoo 17.0 they removed the module `l10n_generic_coa`, but they kept its directory and with only the __init__.py file.

This causes manifest check to fail if `strict` was `True`.

This commit causes `addons_config` to ignore this module for 17.0 and above.
@obayit
Copy link
Contributor Author

obayit commented May 3, 2024

To reproduce the "issue" run this command for a 17.0 deployment:

docker compose run --rm odoo addons init -x --with web

The -x will cause strict to be True.

@pedrobaeza
Copy link
Member

I think the leftover is more a bug than something intentional. I'm on travel for the Spanish OCA days, where I will meet with some people from Odoo SA. I'll check with them about this.

@obayit
Copy link
Contributor Author

obayit commented May 4, 2024

Thanks @pedrobaeza , anyways I created a PR to remove that file from odoo/odoo

@pedrobaeza
Copy link
Member

Closing as the upstream has been merged. Thanks for the follow-up.

@pedrobaeza pedrobaeza closed this May 6, 2024
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.

2 participants