Skip to content

[18.0][MIG] product_category_tax: Migration to 18.0 #2096

Merged
OCA-git-bot merged 25 commits intoOCA:18.0from
ByteMeAsap:18.0-mig-product_category_tax
Jul 24, 2025
Merged

[18.0][MIG] product_category_tax: Migration to 18.0 #2096
OCA-git-bot merged 25 commits intoOCA:18.0from
ByteMeAsap:18.0-mig-product_category_tax

Conversation

@ByteMeAsap
Copy link
Contributor

AaronHForgeFlow and others added 24 commits June 10, 2025 12:57
* add a boleean to exclude specific products to be updated
* Migrate from 13.0
* Synchronize licensing versions
* Move product_template_ids field on product category to
  product_category_product_link module
…eric

Unless it's an override, onchange method names can't be too generic, otherwise
it has a high risk of unadvertedly overwriting another module's method.
1) database searching is much more efficient than py filtering
2) writing on multiple products at once is more efficient
Currently translated at 100.0% (11 of 11 strings)

Translation: account-financial-tools-15.0/account-financial-tools-15.0-product_category_tax
Translate-URL: https://translation.odoo-community.org/projects/account-financial-tools-15-0/account-financial-tools-15-0-product_category_tax/es/
@ByteMeAsap ByteMeAsap force-pushed the 18.0-mig-product_category_tax branch 2 times, most recently from 7992a72 to dbfb9d9 Compare June 10, 2025 10:20
@LoisRForgeFlow
Copy link
Contributor

@ByteMeAsap Dependency has been merged, could you rebase this PR?

@ByteMeAsap ByteMeAsap force-pushed the 18.0-mig-product_category_tax branch from dbfb9d9 to 8e03e5e Compare July 23, 2025 15:01
@ByteMeAsap
Copy link
Contributor Author

@ByteMeAsap Dependency has been merged, could you rebase this PR?

@LoisRForgeFlow , Done !!

Copy link
Contributor

@LoisRForgeFlow LoisRForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional and code review 👍

Copy link
Contributor

@AaronHForgeFlow AaronHForgeFlow left a comment

Choose a reason for hiding this comment

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

Functional ok!

@AaronHForgeFlow
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-2096-by-AaronHForgeFlow-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

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 OCA-git-bot merged commit d0f2124 into OCA:18.0 Jul 24, 2025
9 checks passed
@OCA-git-bot
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants