fix: notifications

avarayr · Sep 20, 2024 · 326f68f · 326f68f
1 parent 6c2b8a7
commit 326f68f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 6 deletions.
diff --git a/src/server/models/Chat.ts b/src/server/models/Chat.ts
@@ -310,11 +310,15 @@ export const Chat = {
           isGenerating: false,
         });
 
-        // Send a notification to the user
-        void WebPush.sendNotification({
-          title: persona.name,
-          message: aiResponse,
-        });
+        try {
+          // Send a notification to the user
+          void WebPush.sendNotification({
+            title: persona.name,
+            message: aiResponse,
+          });
+        } catch (e) {
+          console.error("Error sending notification", e);
+        }
       })();
     } catch (e) {
       console.error("Error generating message in background", e);

diff --git a/src/server/models/WebPush.ts b/src/server/models/WebPush.ts
@@ -25,7 +25,8 @@ export const WebPush = {
   sendNotification: async (message: Notification) => {
     const vapidKeys = await VapidKeys.get();
     if (!vapidKeys) {
-      throw new Error("VAPID keys not found. Please generate them first.");
+      // notifications are disabled
+      return;
     }
 
     const subscriptions = (await db.query("web-push-subscriptions").get<WebPushSubscription>()).getValues();