Skip to content
Snippets Groups Projects

[IMP] domains related to partner_id removed and is_member field added

Merged emanuel buzey requested to merge feature/partner_coordinate_member into dev
3 unresolved threads

This is the link to the Trello card

Merge request reports

Pipeline #52419 passed

Pipeline passed for 2a1f0414 on feature/partner_coordinate_member

Approved by

Merged by emanuel buzeyemanuel buzey 1 year ago (Sep 22, 2023 8:02am UTC)

Merge details

  • Changes merged into dev with 5f1433ae.
  • Deleted the source branch.

Pipeline #52626 passed

Pipeline passed for 5f1433ae on dev

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
31 31 string="Partner",
32 32 )
33 33 effective_date = fields.Date(required=True)
34 is_member = fields.Char(
35 string="Typology", compute="_compute_is_member", readonly=True
36 )
37
38 @api.depends("partner_id.member")
39 def _compute_is_member(self):
40 for record in self:
41 record.is_member = "Socia" if record.partner_id.member else "No socia"
  • emanuel buzey added 1 commit

    added 1 commit

    • 2a1f0414 - [FIX] explicit transaltion added

    Compare with previous version

  • 31 31 string="Partner",
    32 32 )
    33 33 effective_date = fields.Date(required=True)
    34 is_member = fields.Char(
    35 string=_("Typology"), compute="_compute_is_member", readonly=True
  • 5 5 class Inscription(models.Model):
    6 6 _inherit = "energy_project.inscription"
    7 7
    8 partner_id = fields.Many2one(domain=[("member", "=", True)])
    9
    10 @api.constrains("partner_id")
    11 def _check_member(self):
    12 for record in self:
    13 if record.partner_id and not record.partner_id.member:
    14 raise ValidationError(_("The selected partner is not a member"))
    8 partner_id = fields.Many2one()
  • With the changes, looks good to me, push it to dev2 so PO can test it.

  • Daniil Digtyar Vasilieva approved this merge request

    approved this merge request

  • merged

  • emanuel buzey mentioned in commit 5f1433ae

    mentioned in commit 5f1433ae

  • emanuel buzey mentioned in merge request !207 (closed)

    mentioned in merge request !207 (closed)

  • mentioned in merge request !213 (merged)

  • mentioned in merge request !215 (merged)

  • mentioned in merge request !216 (merged)

  • Please register or sign in to reply
    Loading