OCA / multi-company

GNU Affero General Public License v3.0
100 stars 281 forks source link

[17.0][MIG] product_tax_multicompany_default: Migration to version 17.0 #673

Closed carlos-lopez-tecnativa closed 1 month ago

carlos-lopez-tecnativa commented 1 month ago

@Tecnativa TT49882

NOTE: It is necessary to use sudo in test because the account.group_account_manager group does not have permission to create products. See https://github.com/odoo/odoo/commit/512574861691f425ec6a17f20fe4b586bb88a299#r144593878

carolinafernandez-tecnativa commented 1 month ago

I have tested and check this example:

1- I have 2 companies 2- Select with user two companies 3- Create new product selecting two companies with user.

Taxes for first company is added twice

image

If i edit product then it is fixed.

Is it okay this behaviour?

carlos-lopez-tecnativa commented 1 month ago

@carolinafernandez-tecnativa could you please test again?, thanks

pedrobaeza commented 1 month ago

/ocabot migration product_tax_multicompany_default

OCA-git-bot commented 1 month ago

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-673-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot commented 1 month ago

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