[desktop] troubleshot the desktop notifications (not working using th…

…e ARC browser)
nuxeo · Jul 10, 2024 · 8e9f46f · 8e9f46f
1 parent 8625b40
commit 8e9f46f
Showing 1 changed file with 19 additions and 6 deletions.
diff --git a/src/main/desktop-notifier.js b/src/main/desktop-notifier.js
@@ -46,6 +46,7 @@ class DesktopNotifier extends ServiceWorkerComponent {
     return new Promise((resolve, reject) => {
       chrome.notifications.create(namespacedIdOf(id), options, (notificationId) => {
         if (chrome.runtime.lastError) {
+	  // log the error
           console.groupCollapsed('Notification Error');
           console.warn('Failed to create notification');
           console.error(chrome.runtime.lastError);
@@ -58,14 +59,26 @@ class DesktopNotifier extends ServiceWorkerComponent {
           error.info = { id: notificationId, options };
           return reject(error);
         }
+
+	// always log the notification
+        console.groupCollapsed('Created notification');
+        console.info('Notification ID:', notificationId);
+        console.info('Options:', options);
+        console.groupEnd();
+
+	// resolve if no acknowledge required
         if (!options.requireInteraction) {
-          return resolve(notificationId);
+          return resolve(notificationId); 
         }
-        chrome.notifications.onClicked.addListener((clickedId) => {
-          if (clickedId === notificationId) {
-            resolve(notificationId);
-          }
-        });
+
+	// wait for user acknowledge
+        const resolveIfMatchingId = (clickedId) => {
+          if (clickedId !== notificationId) return;
+          resolve(notificationId);
+        };
+        chrome.notifications.onClicked.addListener(resolveIfMatching)
+
+        // let's the caller informed of the asynch condition
         return undefined;
       });
     });