connector-interfaces icon indicating copy to clipboard operation
connector-interfaces copied to clipboard

[18.0][MIG] connector_importer_product: Migration to 18.0

Open vvrossem opened this issue 8 months ago • 3 comments

Notes:

  • Since 17.0, product.packaging::product_id is required: https://github.com/odoo/odoo/blob/17.0/addons/product/models/product_packaging.py#L20

vvrossem avatar Mar 20 '25 10:03 vvrossem

Hello @simahawk, what would you advice to do regarding the CI failing because of multiple WARNING in the logs? :pray:

vvrossem avatar Mar 20 '25 11:03 vvrossem

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

simahawk avatar Mar 20 '25 11:03 simahawk

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). 🤖

OCA-git-bot avatar Mar 27 '25 10:03 OCA-git-bot

@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

Ricardoalso avatar Jul 16 '25 08:07 Ricardoalso

fwd port of https://github.com/OCA/connector-interfaces/pull/153

Ricardoalso avatar Aug 04 '25 09:08 Ricardoalso

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:

  1. review #165
  2. rebase here and start only from the migration commits and the improvements?

simahawk avatar Aug 07 '25 09:08 simahawk

[FIX][18.0] connector_importer: take full control of what are the required_keys #162 isolated in another PR

Ricardoalso avatar Aug 07 '25 10:08 Ricardoalso

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). 🤖

OCA-git-bot avatar Aug 07 '25 10:08 OCA-git-bot

/ocabot merge nobump

simahawk avatar Aug 07 '25 13:08 simahawk

On my way to merge this fine PR! Prepared branch 18.0-ocabot-merge-pr-155-by-simahawk-bump-nobump, awaiting test results.

OCA-git-bot avatar Aug 07 '25 13:08 OCA-git-bot

Congratulations, your PR was merged at e5720ef44d2b27475c5713b3c5d7bd5fd48e0fa7. Thanks a lot for contributing to OCA. ❤️

OCA-git-bot avatar Aug 07 '25 13:08 OCA-git-bot