OCA / edi-framework

GNU Affero General Public License v3.0
6 stars 29 forks source link

[16.0][IMP] edi_oca: add company_id field with multi-company rule #34

Closed OriolMForgeFlow closed 10 months ago

OCA-git-bot commented 10 months ago

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

simahawk commented 10 months ago

IMO

@simahawk Shall we use the default value by default?

I don't think so. The rule is relaxed enough to work by default w/o company. Am I missing any drawback?

etobella commented 10 months ago

It was just a question, it is good for me. Let's wait for the readonly and we can move on with it :smile:

etobella commented 10 months ago

/ocabot merge patch

OCA-git-bot commented 10 months ago

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

OCA-git-bot commented 10 months ago

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