fix: Booking Cancelled Webhook - Organiser webhook payload is missing id. #14697

[vikaspatil0021]

Fixes: #14697

What does this PR do?

fixes the missing id and username information that is available on the Created and Rescheduled webhook payloads.

[vercel]

@vikaspatil0021 is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

[graphite-app]

Graphite Automations

"Add community label" took an action on this PR • (04/25/24)

1 label was added to this PR based on Keith Williams's automation.

"Add consumer team as reviewer" took an action on this PR • (04/25/24)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add foundation team as reviewer" took an action on this PR • (04/26/24)

1 reviewer was added to this PR based on Keith Williams's automation.

[github-actions]

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

[vikaspatil0021]

@alishaz-polymath Please review.

[Udit-takkar]

LGTM. @vikaspatil0021 could you also add this in tests?

[vikaspatil0021]

Hey @Udit-takkar,

cal.com/apps/web/test/utils/bookingScenario/bookingScenario.ts

Line 942 in 8fbb1ed

export function getOrganizer({

cal.com/packages/features/bookings/lib/handleSeats/test/handleSeats.test.ts

Line 1389 in 8fbb1ed

const organizer = getOrganizer({

here organizer structure is different and id already exists. should i add username to them?

[Udit-takkar]

here organizer structure is different and id already exists. should i add username to them?

No not here. check expectBookingCreatedWebhookToHaveBeenFired function in apps/web/test/utils/bookingScenario/expects.ts. create a similar function to check booking cancelled webhook

[vikaspatil0021]

@Udit-takkar Please check

[Udit-takkar]

@vikaspatil0021 use prettier and then push the code. getting lint error in CI

[vikaspatil0021]

@Udit-takkar Please check

[Udit-takkar]

@vikaspatil0021 you don't have to push yarn.lock file in this PR. Please use

git checkout origin/main -- yarn.lock
and then pushing yarn.lock file to undo the changes in yarn.lock file.

2. You haven't added the test. you are not calling the function that you created anywhere.

[Udit-takkar]

LGTM.

TODO: #14886