diff --git a/cooperator/__manifest__.py b/cooperator/__manifest__.py index 6a5f6deed..7248e6368 100644 --- a/cooperator/__manifest__.py +++ b/cooperator/__manifest__.py @@ -17,7 +17,6 @@ "product", "partner_firstname", "partner_contact_birthdate", - "partner_contact_address", "partner_contact_gender", "mail", ], diff --git a/cooperator/models/partner.py b/cooperator/models/partner.py index c25401eac..1a37805e1 100644 --- a/cooperator/models/partner.py +++ b/cooperator/models/partner.py @@ -105,6 +105,9 @@ def _compute_share_info(self): store=True, compute="_compute_representative_of_member_company", ) + # allows for representative to have their own address + # see https://github.com/coopiteasy/vertical-cooperative/issues/350 + type = fields.Selection(selection_add=[("representative", "Representative")]) subscription_request_ids = fields.One2many( "subscription.request", "partner_id", string="Subscription request" ) diff --git a/cooperator/readme/ROADMAP.rst b/cooperator/readme/ROADMAP.rst index 28803e176..76db14f44 100644 --- a/cooperator/readme/ROADMAP.rst +++ b/cooperator/readme/ROADMAP.rst @@ -1,3 +1,8 @@ Gather and consolidate all cooperator settings in the application parameters. +Consider a refactoring for: +1. removing a potential redundancy between the boolean field "representative" and the address type "representative". +2. prevent new contacts to have the representative address type, if they are not representative. +See [this issue](https://github.com/coopiteasy/vertical-cooperative/issues/350) + known_caveats: the cooperator localization modules will have to be splitted in order to install the `cooperator` module without the `cooperator_website` module.