From 22b0644700d3bf1a75608038b9fc18db958603f4 Mon Sep 17 00:00:00 2001 From: Amit Sharma <74371312+Amit91848@users.noreply.github.com> Date: Mon, 15 Jan 2024 20:45:49 +0530 Subject: [PATCH] fix: event reschedule workflow trigger (#13122) * fix event reschedule workflow trigger * uncomment previously failed test * review fix --------- Co-authored-by: Joe Au-Yeung <65426560+joeauyeung@users.noreply.github.com> Co-authored-by: CarinaWolli --- packages/features/bookings/lib/handleNewBooking.ts | 4 ++-- .../bookings/lib/handleNewBooking/test/reschedule.test.ts | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/features/bookings/lib/handleNewBooking.ts b/packages/features/bookings/lib/handleNewBooking.ts index 2ec81f8a5c..a951390e95 100644 --- a/packages/features/bookings/lib/handleNewBooking.ts +++ b/packages/features/bookings/lib/handleNewBooking.ts @@ -2015,7 +2015,7 @@ async function handler( workflows: eventType.workflows, smsReminderNumber: smsReminderNumber || null, calendarEvent: { ...evt, ...{ metadata, eventType: { slug: eventType.slug } } }, - isNotConfirmed: evt.requiresConfirmation || false, + isNotConfirmed: rescheduleUid ? false : evt.requiresConfirmation || false, isRescheduleEvent: !!rescheduleUid, isFirstRecurringEvent: true, emailAttendeeSendToOverride: bookerEmail, @@ -2792,7 +2792,7 @@ async function handler( workflows: eventType.workflows, smsReminderNumber: smsReminderNumber || null, calendarEvent: evtWithMetadata, - isNotConfirmed: !isConfirmedByDefault, + isNotConfirmed: rescheduleUid ? false : !isConfirmedByDefault, isRescheduleEvent: !!rescheduleUid, isFirstRecurringEvent: true, hideBranding: !!eventType.owner?.hideBranding, diff --git a/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts b/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts index b5ce57afbb..6949d44f04 100644 --- a/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts +++ b/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts @@ -850,7 +850,7 @@ describe("handleNewBooking", () => { }, }); - // expectWorkflowToBeTriggered({emails, organizer}); + expectWorkflowToBeTriggered({ emails, organizer }); expectBookingRequestedEmails({ booker,