[16.0][MIG] crm_tag_multi_company #767
Conversation
|
@MarinaAForgeFlow could you squash Migration 16.0 commits? also it would be nice to put in the description of the PR that you are also adding a couple of new modules :) |
MarinaAForgeFlow
force-pushed
the
16.0-mig-crm_tag_multi_company
branch
from
1dc34c9 to
ed5c1a1
Compare
MarinaAForgeFlow
changed the title
| <field name="arch" type="xml"> | ||
| <field name="name" position="after"> | ||
| <field name="company_id" groups="base.group_multi_company" /> | ||
| <field name="company_id" invisible="1" /> |
There was a problem hiding this comment.
Is this second line mandatory ?
it is not explained in the odoo documentation.
https://www.odoo.com/documentation/18.0/fr/developer/howtos/company.html#views
There was a problem hiding this comment.
I understood it was needed after reading this point on the migration notes for v16.0 (https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-16.0):
On the same mood, you should check if the fields protected by a group interact in some way with business logic or UI. For example, a field being part of a domain of another field, or being used in copy/defaults. An special case is the field company_id, which is always used internally for domains on many2one fields referencing multi-company aware records. On these cases, you have to add the same field twice, one with the group, and another invisible without it.
|
/ocabot merge nobump |
|
Hey, thanks for contributing! Proceeding to merge this for you. |
|
Congratulations, your PR was merged at ac41464. Thanks a lot for contributing to OCA. ❤️ |
Supersedes PR #659
[ADD] crm_tag_multi_company_sale: glue module
[ADD] crm_tag_multi_company_event_crm: glue module