search

OCA / multi-company

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

[ADD] sale_product_company_multi_add #587

Open santostelmo opened 7 months ago

santostelmo commented 7 months ago

sale_product_company allows you to select a list of companies in which you can sell a product. A product can be sold only in the companies defined in the field Selling Companies. ▶️ Depends on https://github.com/OCA/multi-company/pull/584

That module apply that principle when adding multiple products with sale_product_multi_add

leemannd commented 6 months ago

@cyrilmanuel The failing test is one that checks the dependencies. It comes from -> https://github.com/OCA/multi-company/pull/587/files#diff-fac4c6890301d4de5c3f4266837803d5240c84a3d8b6c735bbc6a64c39d2f94eR2

Once we merge it and remove the test dependency it will be green as cloverfield

cyrilmanuel commented 6 months ago

@pedrobaeza can you review / merge this PR please ?

pedrobaeza commented 6 months ago

CI is red

santostelmo commented 5 months ago

CI green now

OCA-git-bot commented 5 months ago

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

cyrilmanuel commented 3 months ago

Hi @pedrobaeza can you merge this PR ?

gurneyalex commented 3 months ago

/ocabot merge nobump

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-587-by-gurneyalex-bump-nobump, awaiting test results.

OCA-git-bot commented 3 months ago

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

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.