Make sure all promises errors are caught in scheduled function and schedule it every 5 mins

Author: pauljohanneskraft

functions/src/functions/onSchedule.ts

@@ -13,7 +13,7 @@ import { getServiceFactory } from "../services/factory/getServiceFactory.js";
 
 export const onScheduleEveryMorning = onSchedule(
   {
-    schedule: "0 8 * * *",
+    schedule: "*/5 * * * *",
     timeZone: "America/Los_Angeles",
     serviceAccount: privilegedServiceAccount,
     secrets: Env.twilioSecretKeys,

functions/src/services/trigger/triggerService.ts

@@ -63,17 +63,41 @@ export class TriggerService {
         patients,
         messageService,
         now,
-      }),
+      }).catch((error) =>
+        logger.error(
+          `everyMorning: Error adding daily reminders: ${String(error)}`,
+        ),
+      ),
       this.addInactivityReminderMessages({
         patients,
         now,
         messageService,
         userService,
-      }),
-      this.addAppointmentReminderMessages(now),
-      this.completeAppointmentReminderMessages(now),
-      this.seedStaticDataIfNeeded(),
-      this.deleteExpiredAccounts(),
+      }).catch((error) =>
+        logger.error(
+          `everyMorning: Error adding inactivity reminders: ${String(error)}`,
+        ),
+      ),
+      this.addAppointmentReminderMessages(now).catch((error) =>
+        logger.error(
+          `everyMorning: Error adding appointment reminders: ${String(error)}`,
+        ),
+      ),
+      this.completeAppointmentReminderMessages(now).catch((error) =>
+        logger.error(
+          `everyMorning: Error completing appointment reminders: ${String(error)}`,
+        ),
+      ),
+      this.seedStaticDataIfNeeded().catch((error) =>
+        logger.error(
+          `everyMorning: Error seeding static data: ${String(error)}`,
+        ),
+      ),
+      this.deleteExpiredAccounts().catch((error) =>
+        logger.error(
+          `everyMorning: Error deleting expired accounts: ${String(error)}`,
+        ),
+      ),
     ]);
   }