diff --git a/cooperator/__manifest__.py b/cooperator/__manifest__.py index b03bafd0f..c58b8cf72 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 7dfff1f0d..326eebf34 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 15d0bfc92..e7e0c0cca 100644 --- a/cooperator/readme/ROADMAP.rst +++ b/cooperator/readme/ROADMAP.rst @@ -1 +1,6 @@ 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)