allow notifications even without an ID

project.task.yaml

@@ -4,6 +4,11 @@ tasks:
 - 08 notifications :
   - get the rest of our Android devices to work... and insert tooling (exportable logs?) so that we can see problems and troubleshoot as we onboard
     - get an error registering notifications on Firefox and subscription info is null
+    - if navigator.serviceWorker is null, then tell the user to wait
+    - Android DuckDuckGo asked for my permissions, got error, won't download DB
+    - Android Chrome won't ask permission, will download log but always empty
+    -
+  - Chrome will get notification permissions but still complain when I reload (maybe I reload too soon?)
   - prompt user to install on their home screen https://benborgers.com/posts/pwa-detect-installed
   - warn if they're using the web (android only?)
     https://developer.mozilla.org/en-US/docs/Web/API/Navigator/getInstalledRelatedApps

src/views/HelpNotificationsView.vue

@@ -22,7 +22,7 @@
     </div>
 
     <div>
-      <p>Here are things to try to get notifications working.</p>
+      <p>Here are things to get notifications working.</p>
 
       <h2 class="text-xl font-semibold">Test</h2>
       <p>Somehow call the service-worker self.showNotification</p>
@@ -54,7 +54,10 @@
           <div>Make sure your OS (above) supports it.</div>
 
           <h3 class="text-lg font-semibold">Mobile Phone - Android</h3>
-          <div>Chrome requires version 50.</div>
+          <div>
+            Chrome requires version 50. Hold icon, select "App info", and enable
+            notifications.
+          </div>
         </div>
       </div>
 

sw_scripts/safari-notifications.js

@@ -461,10 +461,10 @@ function updateRecord(store, data) {
 
 async function fetchAllAccounts() {
   return new Promise((resolve, reject) => {
-    let openRequest = indexedDB.open("TimeSafariAccounts");
+    const openRequest = indexedDB.open("TimeSafariAccounts");
 
     openRequest.onupgradeneeded = function (event) {
-      let db = event.target.result;
+      const db = event.target.result;
       if (!db.objectStoreNames.contains("accounts")) {
         db.createObjectStore("accounts", { keyPath: "id" });
       }
@@ -474,10 +474,10 @@ async function fetchAllAccounts() {
     };
 
     openRequest.onsuccess = function (event) {
-      let db = event.target.result;
-      let transaction = db.transaction("accounts", "readonly");
-      let objectStore = transaction.objectStore("accounts");
-      let getAllRequest = objectStore.getAll();
+      const db = event.target.result;
+      const transaction = db.transaction("accounts", "readonly");
+      const objectStore = transaction.objectStore("accounts");
+      const getAllRequest = objectStore.getAll();
 
       getAllRequest.onsuccess = function () {
         resolve(getAllRequest.result);
@@ -494,12 +494,8 @@ async function fetchAllAccounts() {
 }
 
 async function getNotificationCount() {
-  let secret = null;
   let accounts = [];
   let result = null;
-  if ("secret" in self) {
-    secret = self.secret;
-    const secretUint8Array = self.decodeBase64(secret);
   // 1 is our master settings ID; see MASTER_SETTINGS_KEY
   const settings = await getSettingById(1);
   let lastNotifiedClaimId = null;
@@ -507,49 +503,58 @@ async function getNotificationCount() {
     lastNotifiedClaimId = settings["lastNotifiedClaimId"];
   }
   const activeDid = settings["activeDid"];
+  console.log("safari-not getNotificationsCount last", lastNotifiedClaimId);
   accounts = await fetchAllAccounts();
-    let did = null;
-    for (var i = 0; i < accounts.length; i++) {
-      let account = accounts[i];
-      let did = account["did"];
-      if (did == activeDid) {
-        let publicKeyHex = account["publicKeyHex"];
-        let identity = account["identity"];
+  let activeAccount = null;
+  for (let i = 0; i < accounts.length; i++) {
+    if (accounts[i]["did"] == activeDid) {
+      activeAccount = accounts[i];
+      break;
+    }
+  }
+
+  const headers = {
+    "Content-Type": "application/json",
+  };
+
+  const identity = activeAccount && activeAccount["identity"];
+  if (identity && "secret" in self) {
+    const secret = self.secret;
+    const secretUint8Array = self.decodeBase64(secret);
     const messageWithNonceAsUint8Array = self.decodeBase64(identity);
     const nonce = messageWithNonceAsUint8Array.slice(0, 24);
     const message = messageWithNonceAsUint8Array.slice(24, identity.length);
     const decoder = new TextDecoder("utf-8");
     const decrypted = self.secretbox.open(message, nonce, secretUint8Array);
-
     const msg = decoder.decode(decrypted);
     const identifier = JSON.parse(JSON.parse(msg));
 
-        const headers = {
-          "Content-Type": "application/json",
-        };
-
     headers["Authorization"] = "Bearer " + (await accessToken(identifier));
+  }
 
-        let response = await fetch(
+  console.log("safari-not getNotificationsCount fetch", headers);
+  const response = await fetch(
     settings["apiServer"] + "/api/v2/report/claims",
     {
       method: "GET",
       headers: headers,
     },
   );
+  console.log("safari-not getNotificationsCount json", response);
   if (response.status == 200) {
-          let json = await response.json();
-          let claims = json["data"];
+    const json = await response.json();
+    console.log("safari-not getNotificationsCount json", json);
+    const claims = json["data"];
     let newClaims = 0;
-          for (var i = 0; i < claims.length; i++) {
-            let claim = claims[i];
+    for (let i = 0; i < claims.length; i++) {
+      const claim = claims[i];
       if (claim["id"] === lastNotifiedClaimId) {
         break;
       }
       newClaims++;
     }
     if (newClaims > 0) {
-            result = `There are ${newClaims} new activities on TimeSafari`;
+      result = `There are ${newClaims} new activities on Time Safari`;
     }
     const most_recent_notified = claims[0]["id"];
     await setMostRecentNotified(most_recent_notified);
@@ -560,10 +565,7 @@ async function getNotificationCount() {
       response,
     );
   }
-        break;
-      }
-    }
-  }
+
   return result;
 }