search

OCA / multi-company

GNU Affero General Public License v3.0
98 stars 280 forks source link

[15.0][FIX] purchase_sale_inter_company: sync confirmed unlocked purchase upadtes #561

Closed chienandalu closed 6 months ago

chienandalu commented 7 months ago

fw from

When the purchase user updates the PO after the SO is generated and always if both are confirmed and unlocked, we sync the new lines to update the demand.

cc @Tecnativa TT46773

please review @pedrobaeza

@francesco-ooops @aleuffre @renda-dev I started forwarding your 14.0 prs but the task isn't trivial as the module was split in v15

francesco-ooops commented 6 months ago

@chienandalu I tried testing on runboat, installing the module

do you know why there is only one company in the instance?

chienandalu commented 6 months ago

do you know why there is only one company in the instance?

I guess you used the base runboat. If you use the full one, you'll find all the demo data.

francesco-ooops commented 6 months ago

@chienandalu ok, unrelated to this PR, but what module/application are the multi-company demo data related to?

chienandalu commented 6 months ago

I think that the demo data for Chicago goes with stock

pedrobaeza commented 6 months ago

/ocabot merge minor

OCA-git-bot commented 6 months ago

On my way to merge this fine PR! Prepared branch 15.0-ocabot-merge-pr-561-by-pedrobaeza-bump-minor, awaiting test results.

OCA-git-bot commented 6 months ago

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