Skip to content

[18.0][IMP] partner_event: avoid matching partner_id as attendee when email…#509

Open
cvinh wants to merge 1 commit into
OCA:18.0from
invitu:18.0-imp_partner_event_attendee_no_email
Open

[18.0][IMP] partner_event: avoid matching partner_id as attendee when email…#509
cvinh wants to merge 1 commit into
OCA:18.0from
invitu:18.0-imp_partner_event_attendee_no_email

Conversation

@cvinh

@cvinh cvinh commented Jun 29, 2026

Copy link
Copy Markdown

… is shared

Steps to reproduce :

  1. Install partner_event
  2. Create an event with "Create Partners in registration" checked
  3. Create a partner with an email address
  4. Create a registration for that event with:
    • partner_id = the partner above
    • name = "Child Attendee" (a different person)
    • email = same email as partner_id (e.g. fallback when attendee has no email)
  5. Check attendee_partner_id on the registration

Expected: attendee_partner_id is created with empty email
Got: attendee_partner_id = partner_id

@cvinh cvinh marked this pull request as ready for review June 29, 2026 02:17
@cvinh cvinh force-pushed the 18.0-imp_partner_event_attendee_no_email branch 3 times, most recently from 65e9df8 to a5331c6 Compare June 29, 2026 04:35
@cvinh cvinh marked this pull request as draft June 29, 2026 04:39
@cvinh cvinh force-pushed the 18.0-imp_partner_event_attendee_no_email branch 2 times, most recently from 211a263 to a8d6bf9 Compare June 29, 2026 05:15
@cvinh cvinh force-pushed the 18.0-imp_partner_event_attendee_no_email branch from a8d6bf9 to 9f28f38 Compare June 29, 2026 05:37
@cvinh cvinh marked this pull request as ready for review June 29, 2026 06:06
@pedrobaeza pedrobaeza added this to the 18.0 milestone Jun 29, 2026

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants