Hotfix/fix workflows (#6071)

* Hotfix: metadata parsing * Hotfix: removed data64 avatar for 4mb payload limit * Hotfix: Installed Apps Tabs Switchi g (#5063) (#5064) * Hotfix: metadata parsing * Use Router query as tabs use shallow routing, data-fetching doesnt occur on switching tabs Co-authored-by: zomars <zomars@me.com> Co-authored-by: zomars <zomars@me.com> * Hotfix/email scheduling memory limit (#5839) * Hotfix: metadata parsing * Hotfix: removed data64 avatar for 4mb payload limit * Hotfix: Installed Apps Tabs Switchi g (#5063) (#5064) * Hotfix: metadata parsing * Use Router query as tabs use shallow routing, data-fetching doesnt occur on switching tabs Co-authored-by: zomars <zomars@me.com> Co-authored-by: zomars <zomars@me.com> * remove if and at to query Co-authored-by: zomars <zomars@me.com> Co-authored-by: Hariom Balhara <hariombalhara@gmail.com> Co-authored-by: CarinaWolli <wollencarina@gmail.com> * Fix/workflow email (#6069) * teams plan needed * disable * disable * type fix Co-authored-by: CarinaWolli <wollencarina@gmail.com> Co-authored-by: zomars <zomars@me.com> Co-authored-by: Hariom Balhara <hariombalhara@gmail.com> Co-authored-by: CarinaWolli <wollencarina@gmail.com>
calcom · Dec 16, 2022 · 53fd407 · 53fd407 · vercel · Dec 16, 2022
1 parent 962b463
commit 53fd407
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 13 deletions.
diff --git a/packages/features/ee/workflows/lib/reminders/emailReminderManager.ts b/packages/features/ee/workflows/lib/reminders/emailReminderManager.ts
@@ -39,6 +39,7 @@ export const scheduleEmailReminder = async (
   workflowStepId: number,
   template: WorkflowTemplates
 ) => {
+  if (action === WorkflowActions.EMAIL_ADDRESS) return;
   const { startTime, endTime } = evt;
   const uid = evt.uid as string;
   const currentDate = dayjs();
@@ -76,10 +77,6 @@ export const scheduleEmailReminder = async (
       attendeeName = evt.organizer.name;
       timeZone = evt.attendees[0].timeZone;
       break;
-    case WorkflowActions.EMAIL_ADDRESS:
-      name = "";
-      attendeeName = evt.attendees[0].name;
-      timeZone = evt.organizer.timeZone;
   }
 
   let emailContent = {

diff --git a/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts b/packages/features/ee/workflows/lib/reminders/reminderScheduler.ts
@@ -58,8 +58,7 @@ export const scheduleWorkflowReminders = async (
             );
           } else if (
             step.action === WorkflowActions.EMAIL_ATTENDEE ||
-            step.action === WorkflowActions.EMAIL_HOST ||
-            step.action === WorkflowActions.EMAIL_ADDRESS
+            step.action === WorkflowActions.EMAIL_HOST
           ) {
             let sendTo = "";
 
@@ -70,8 +69,6 @@ export const scheduleWorkflowReminders = async (
               case WorkflowActions.EMAIL_ATTENDEE:
                 sendTo = evt.attendees[0].email;
                 break;
-              case WorkflowActions.EMAIL_ADDRESS:
-                sendTo = step.sendTo || "";
             }
             scheduleEmailReminder(
               evt,
@@ -129,8 +126,7 @@ export const sendCancelledReminders = async (
             );
           } else if (
             step.action === WorkflowActions.EMAIL_ATTENDEE ||
-            step.action === WorkflowActions.EMAIL_HOST ||
-            step.action === WorkflowActions.EMAIL_ADDRESS
+            step.action === WorkflowActions.EMAIL_HOST
           ) {
             let sendTo = "";
 
@@ -141,8 +137,6 @@ export const sendCancelledReminders = async (
               case WorkflowActions.EMAIL_ATTENDEE:
                 sendTo = evt.attendees[0].email;
                 break;
-              case WorkflowActions.EMAIL_ADDRESS:
-                sendTo = step.sendTo || "";
             }
             scheduleEmailReminder(
               evt,

diff --git a/packages/trpc/server/routers/viewer/workflows.tsx b/packages/trpc/server/routers/viewer/workflows.tsx
@@ -884,7 +884,7 @@ export const workflowsRouter = router({
           throw new TRPCError({ code: "UNAUTHORIZED" });
         }
 
-        if (isSMSAction(step.action)) {
+        if (isSMSAction(step.action) || step.action === WorkflowActions.EMAIL_ADDRESS) {
           const hasTeamPlan = (await ctx.prisma.membership.count({ where: { userId: user.id } })) > 0;
           if (!hasTeamPlan) {
             throw new TRPCError({ code: "UNAUTHORIZED", message: "Team plan needed" });