fix: metadata for seats in webhooks (calcom#14025)

* add metadata to seats for workflows and webhooks

* add test

---------

Co-authored-by: CarinaWolli <[email protected]>
Co-authored-by: Alex van Andel <[email protected]>

Author: 3 people

packages/features/bookings/lib/handleNewBooking.ts

@@ -864,6 +864,7 @@ export const findBookingQuery = async (bookingId: number) => {
       description: true,
       status: true,
       responses: true,
+      metadata: true,
       user: {
         select: {
           name: true,

packages/features/bookings/lib/handleSeats/handleSeats.ts

@@ -96,8 +96,10 @@ const handleSeats = async (newSeatedBookingObject: NewSeatedBookingObject) => {
 
   // If the resultBooking is defined we should trigger workflows else, trigger in handleNewBooking
   if (resultBooking) {
-    // Obtain event metadata that includes videoCallUrl
-    const metadata = evt.videoCallData?.url ? { videoCallUrl: evt.videoCallData.url } : undefined;
+    const metadata = {
+      ...(typeof resultBooking.metadata === "object" && resultBooking.metadata),
+      ...reqBodyMetadata,
+    };
     try {
       await scheduleWorkflowReminders({
         workflows: eventType.workflows,
@@ -126,7 +128,7 @@ const handleSeats = async (newSeatedBookingObject: NewSeatedBookingObject) => {
       rescheduleEndTime: originalRescheduledBooking?.endTime
         ? dayjs(originalRescheduledBooking?.endTime).utc().format()
         : undefined,
-      metadata: { ...metadata, ...reqBodyMetadata },
+      metadata,
       eventTypeId,
       status: "ACCEPTED",
       smsReminderNumber: seatedBooking?.smsReminderNumber || undefined,

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

@@ -187,7 +187,9 @@ describe("handleSeats", () => {
         body: mockBookingData,
       });
 
-      await handleNewBooking(req);
+      const createdBooking = await handleNewBooking(req);
+
+      expect(createdBooking.metadata).toHaveProperty("videoCallUrl");
 
       const handleSeatsCall = spy.mock.calls[0][0];