Skip to content

[MSD benchmark] fix: handle collective multiple host on destinationCalendar#3

Open
khaliqgant wants to merge 14 commits into
msd-benchmark-base/cal_dot_com/pr-10967from
msd-benchmark/cal_dot_com/pr-10967
Open

[MSD benchmark] fix: handle collective multiple host on destinationCalendar#3
khaliqgant wants to merge 14 commits into
msd-benchmark-base/cal_dot_com/pr-10967from
msd-benchmark/cal_dot_com/pr-10967

Conversation

@khaliqgant

Copy link
Copy Markdown

MSD benchmark PR recreated from Martian Code Review Bench.

Source PR: calcom#10967
Dataset: cal_dot_com
Original title: fix: handle collective multiple host on destinationCalendar

What does this PR do?

  • Converts destinationCalendar on CalendarEvent to be an array.
  • Patches and fixes so every sentence of code that was using evet.destinationCalendar can support the new type of array.
  • The logic here is to fallback to normal behaviour has having one main destinationCalendar but also supports multiple so host can get invited correctly.

Fixes calcom#7754

Loom

Creating event and rescheduling:
https://www.loom.com/share/3cb3af513d274ed2bdfed44e48edddf6?sid=d81ce3de-2893-4a30-b085-ff67c31b7dfb

Canceling event:
https://www.loom.com/share/7ef505d4665e4d4982d2bb2d7b118085?sid=cd43c97c-aa86-4667-979f-f6492adf3365

Requirement/Documentation

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How should this be tested?

  • Are there environment variables that should be set? No
  • What are the minimal test data to have? Setup a collective event using teampro and teamfree users. Add any app calendar to those accounts and add them as host for collective event.
  • What is expected (happy path) to have (input and output)? Collective event with 2 host and every user with any calendar app installed should receive proper calendar events.
  • Any other important info that could help to test that PR

Mandatory Tasks

  • Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

  • I haven't added tests that prove my fix is effective or that my feature works

@khaliqgant khaliqgant changed the base branch from main to msd-benchmark-base/cal_dot_com/pr-10967 May 15, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CAL-1267] Collective Events have only one team attendee on the google event

2 participants