search

Tecnativa / doodba

Base image for making the creation of customized Odoo environments a piece of cake
Apache License 2.0
423 stars 302 forks source link

Ignore manifest check for l10n_generic_coa in 17.0 and above #609

Closed obayit closed 4 months ago

obayit commented 4 months ago

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 commented 4 months ago

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 commented 4 months ago

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 commented 4 months ago

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

pedrobaeza commented 4 months ago

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