[18.0][MIG] connector_importer_product: Migration to 18.0
Notes:
- Since 17.0,
product.packaging::product_idisrequired: https://github.com/odoo/odoo/blob/17.0/addons/product/models/product_packaging.py#L20
Hello @simahawk, what would you advice to do regarding the CI failing because of multiple WARNING in the logs? :pray:
silent the ones that you can in the tests (eg: the warning for debug mode). This we can even turn it off when in test mode directly (see where debug_mode is used).
For the others, if any you can ignore the via https://github.com/OCA/connector-interfaces/blob/18.0/checklog-odoo.cfg
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
@vvrossem I just rebased this PR in order to be able to migrate this pending.merge https://github.com/OCA/connector-interfaces/pull/141/commits
fwd port of https://github.com/OCA/connector-interfaces/pull/153
As I need a clear picture on the changes I've decided to add the module as not installable to v18. See https://github.com/OCA/connector-interfaces/pull/166.
Can you please:
- review #165
- rebase here and start only from the migration commits and the improvements?
[FIX][18.0] connector_importer: take full control of what are the required_keys #162 isolated in another PR
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
/ocabot merge nobump
On my way to merge this fine PR! Prepared branch 18.0-ocabot-merge-pr-155-by-simahawk-bump-nobump, awaiting test results.
Congratulations, your PR was merged at e5720ef44d2b27475c5713b3c5d7bd5fd48e0fa7. Thanks a lot for contributing to OCA. ❤️