fix: Rescheduling email when there is broken calendar integration (ca…

…lcom#15669)
p6l-richard · Jul 22, 2024 · 6baf1dc · 6baf1dc
1 parent 874c3fc
commit 6baf1dc
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/packages/features/bookings/lib/handleNewBooking.ts b/packages/features/bookings/lib/handleNewBooking.ts
@@ -1268,8 +1268,7 @@ async function handler(
 
  evt.appsStatus = handleAppsStatus(results, booking, reqAppsStatus);
 
- // If there is an integration error, we don't send successful rescheduling email, instead broken integration email should be sent that are handled by either CalendarManager or videoClient
- if (noEmail !== true && isConfirmedByDefault && !isThereAnIntegrationError) {
+ if (noEmail !== true && isConfirmedByDefault) {
  const copyEvent = cloneDeep(evt);
  const copyEventAdditionalInfo = {
  ...copyEvent,

diff --git a/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts b/packages/features/bookings/lib/handleNewBooking/test/reschedule.test.ts
@@ -660,8 +660,11 @@ describe("handleNewBooking", () => {
 
  expectWorkflowToBeTriggered({ emailsToReceive: [organizer.email], emails });
 
- // FIXME: We should send Broken Integration emails on calendar event updation failure
- // expectBrokenIntegrationEmails({ booker, organizer, emails });
+ expectSuccessfulBookingRescheduledEmails({
+ booker,
+ organizer,
+ emails,
+ });
 
  expectBookingRescheduledWebhookToHaveBeenFired({
  booker,