-
-
Notifications
You must be signed in to change notification settings - Fork 873
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ADD] sale_partner_company_group #1247
[ADD] sale_partner_company_group #1247
Conversation
2b233fd
to
f98d007
Compare
49a23e0
to
777c132
Compare
There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. |
777c132
to
f5dbec6
Compare
f5dbec6
to
67fd228
Compare
file no longer required now that the base PR is merged.
Co-authored-by: Akim Juillerat <akim.juillerat@camptocamp.com>
Co-authored-by: Akim Juillerat <akim.juillerat@camptocamp.com>
/ocabot merge nobump |
This PR looks fantastic, let's merge it! |
It looks like something changed on |
Congratulations, your PR was merged at 2727ee1. Thanks a lot for contributing to OCA. ❤️ |
The idea is to split
partner_company_group
to several modules depending on each of core modules. This is needed to avoid installing unwanted dependencies on the project, for example we needpartner_company_group
, but without additional installedaccount
,sale
orcrm
.So, finally we want it to be:
base_partner_company_group
-> [15.0][ADD] base_partner_company_group #1244account_partner_company_group
depending onbase_partner_company_group
andaccount
-> [ADD] account_partner_company_group #1246crm_partner_company_group
depending onbase_partner_company_group
andcrm
-> [ADD] crm_partner_company_group #1245sale_partner_company_group
depending onbase_partner_company_group
andsale
-> [ADD] sale_partner_company_group #1247