Always open notification

blakeblackshear · Jul 21, 2024 · c92c825 · c92c825
1 parent 367555b
commit c92c825
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 18 deletions.
diff --git a/frigate/comms/webpush.py b/frigate/comms/webpush.py
@@ -90,7 +90,7 @@ def send_message(self, payload: dict[str, any]) -> None:
 
         title = f"{', '.join(sorted_objects).replace('_', ' ').title()}{' was' if state == 'end' else ''} detected in {', '.join(payload['after']['data']['zones']).replace('_', ' ').title()}"
         message = f"Detected on {payload['after']['camera'].replace('_', ' ').title()}"
-        link = f"/review?id={reviewId}"
+        direct_url = f"/review?id={reviewId}"
         image = f'{payload["after"]["thumb_path"].replace("/media/frigate", "")}'
 
         for pusher in self.web_pushers:
@@ -101,7 +101,7 @@ def send_message(self, payload: dict[str, any]) -> None:
                     {
                         "title": title,
                         "message": message,
-                        "link": link,
+                        "direct_url": direct_url,
                         "image": image,
                         "id": reviewId,
                     }

diff --git a/web/public/notifications-worker.js b/web/public/notifications-worker.js
@@ -29,24 +29,11 @@ self.addEventListener("notificationclick", (event) => {
     // @ts-expect-error we know this exists
     if (event.notification.data) {
       const url = event.notification.data.link;
-
       // eslint-disable-next-line no-undef
-      clients.matchAll({ type: "window" }).then((windowClients) => {
-        // Check if there is already a window/tab open with the target URL
-        for (let i = 0; i < windowClients.length; i++) {
-          const client = windowClients[i];
-          // If so, just focus it.
-          if (client.url === url && "focus" in client) {
-            return client.focus();
-          }
-        }
-        // If not, then open the target URL in a new window/tab.
+      if (clients.openWindow) {
         // eslint-disable-next-line no-undef
-        if (clients.openWindow) {
-          // eslint-disable-next-line no-undef
-          return clients.openWindow(url);
-        }
-      });
+        return clients.openWindow(url);
+      }
     }
   }
 });