Merge pull request #105 from dvishal485/main

update notif send logic
dtutimes · Aug 26, 2024 · 4f30d2f · 4f30d2f
2 parents 39d923b + 99dfe65
commit 4f30d2f
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/src/api/controllers/notificationController.ts b/src/api/controllers/notificationController.ts
@@ -29,9 +29,17 @@ export const push_notification = async () => {
     PRIVATE_KEY,
   );
 
-  for (const subscription of subscriptions) {
-    await webpush.sendNotification(subscription, notificationPayload);
-  }
+  subscriptions.forEach((subscription) => {
+    webpush
+      .sendNotification(subscription, notificationPayload)
+      .then(() => {
+        console.log("Notification sent");
+      })
+      .catch(async (err) => {
+        console.error("Error sending notification", err);
+        await subscription.deleteOne();
+      });
+  });
 };
 
 export const test_notif = asyncErrorHandler(async (req, res, _next) => {