fix: seated event bug

[Udit-takkar]

What does this PR do?

• Fixes #XXXX (GitHub issue number)
• Fixes CAL-XXXX (Linear issue number - should be visible at the bottom of the GitHub issue description)

Visual Demo (For contributors especially)

A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).

Video Demo (if applicable):

• Show screen recordings of the issue or feature.
• Demonstrate how to reproduce the issue, the behavior before and after the change.

Image Demo (if applicable):

• Add side-by-side screenshots of the original and updated change.
• Highlight any significant change(s).

Mandatory Tasks (DO NOT REMOVE)

• I have self-reviewed the code (A decent size PR without self-review might be rejected).
• I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
• I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

• Are there environment variables that should be set?
• What are the minimal test data to have?
• What is expected (happy path) to have (input and output)?
• Any other important info that could help to test that PR

Checklist

• I haven't read the contributing guide
• My code doesn't follow the style guidelines of this project
• I haven't commented my code, particularly in hard-to-understand areas
• I haven't checked if my changes generate no new warnings

---

Summary by cubic

Fixes attendee visibility in reassignment emails for seated events. Reassignment notifications now show or hide the attendee list based on the event’s seatsShowAttendees setting.

• Bug Fixes
  • Pass showAttendees to AttendeeUpdatedEmail via sendReassignedUpdatedEmailsAndSMS; include seatsPerTimeSlot in the reassignment payload to flag seated events.
  • Set showAttendees from the event type during reassignment across round-robin and managed-event flows.

^{Written for commit 416cd24. Summary will update on new commits.}

[anikdhabal]

Looks good

[cubic-dev-ai]

No issues found across 5 files

[hariombalhara]

What is the reason to add this change as well here? is this fixing something else too?

[hariombalhara]

nvm, found the reason, it is being used down the flow and without it seatsShowAttendees false wont actually jhave an effect

[hariombalhara]

LGTM !!

[hariombalhara]

@Udit-takkar @anikdhabal Devin says this. Looks like a bug

In ManagedEventManualReassignmentService.ts:700-705, the withEventType call does NOT include seatsPerTimeSlot:
.withEventType({
id: targetEventTypeDetails.id,
slug: targetEventTypeDetails.slug,
description: newBooking.description,
schedulingType: parentEventType.schedulingType,
})
The filtering logic in AttendeeScheduledEmail requires BOTH conditions:
if (!showAttendees && calEvent.seatsPerTimeSlot) { ... }
Impact: The fix adds showAttendees: !!targetEventTypeDetails.seatsShowAttendees but without seatsPerTimeSlot on the calEvent, the filtering won't execute. The fix works for round-robin reassignments but may not work for managed event reassignments.

[hariombalhara]

Blocking till, we get confirmation on possible bug higlighted by Devin

[github-actions]

E2E results are ready!

• Workflow #99472.2 latest results

[anikdhabal]

@hariombalhara good find, missed it

[cubic-dev-ai]

No issues found across 6 files