search

OCA / multi-company

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

[16.0][IMP] product_account_multicompany_default propagate product category accounts #650

Open santostelmo opened 3 months ago

santostelmo commented 3 months ago

Propagate product category accounts from the current company to all other companies where you have access rights.

OCA-git-bot commented 3 months ago

Hi @yajo, some modules you are maintaining are being modified, check this out!

santostelmo commented 3 months ago

Thanks for the contribution. It looks pretty well. However, I can see no changes on README.

Since this impacts in the module functionality, please extend the description and usage sections. This way we'll know better how to test this functionally.

Thanks for your review @yajo . I have updated README mentioning propagation of accounting fields on product category.

cyrilmanuel commented 3 months ago

Hi @yajo it's Ok for you ?? can you merge this PR :)

cyrilmanuel commented 3 months ago

Hello @pedrobaeza, could you merge this PR ?

pedrobaeza commented 3 months ago

/ocabot merge major

OCA-git-bot commented 3 months ago

Hey, thanks for contributing! Proceeding to merge this for you. Prepared branch 16.0-ocabot-merge-pr-650-by-pedrobaeza-bump-major, awaiting test results.

OCA-git-bot commented 3 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-650-by-pedrobaeza-bump-major.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

OCA-git-bot commented 3 months ago

What a great day to merge this nice PR. Let's do it! Prepared branch 16.0-ocabot-merge-pr-650-by-yajo-bump-minor, awaiting test results.

OCA-git-bot commented 3 months ago

@yajo your merge command was aborted due to failed check(s), which you can inspect on this commit of 16.0-ocabot-merge-pr-650-by-yajo-bump-minor.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

gurneyalex commented 2 months ago

@santostelmo can you check the failures on merge?