fix: ios push notifications

Author: Soxasora

sw/eventListener.js

@@ -11,6 +11,7 @@ const storage = new ServiceWorkerStorage('sw:storage', 1)
 let messageChannelPort
 let actionChannelPort
 
+// TODO 756.2 Fix iOS check, initialValue isn't updated if iOS is true
 // operating system. the value will be received via a STORE_OS message from app since service workers don't have access to window.navigator
 let os = ''
 const iOS = () => os === 'iOS'
@@ -131,7 +132,7 @@ const mergeAndShowNotification = async (sw, payload, currentNotifications, tag,
     }
     const newPayload = { ...data, amount: newAmount, sats: newSats }
     return newPayload
-  }, { ...incomingData, amount: initialAmount })
+  }, { ...incomingData, amount: 1 }) // 1 is hard coded to test iOS scenario, should be initialValue
 
   log(`[sw:push] ${nid} - merged payload: ${JSON.stringify(mergedPayload)}`)
 
@@ -142,7 +143,7 @@ const mergeAndShowNotification = async (sw, payload, currentNotifications, tag,
     if (compareTag === 'REPLY') {
       title = `you have ${amount} new replies`
     } else if (compareTag === 'MENTION') {
-      title = `you were mentioned ${amount} times`
+      title = `you were mentioned ${amount} times` // PROPOSAL: url should be notifications page
     } else if (compareTag === 'ITEM_MENTION') {
       title = `your items were mentioned ${amount} times`
     } else if (compareTag === 'REFERRAL') {
@@ -175,6 +176,9 @@ const mergeAndShowNotification = async (sw, payload, currentNotifications, tag,
   return await sw.registration.showNotification(title, options)
 }
 
+// TODO 756.3 sometimes you get sent to very old threads, this is because the notification is not updated
+// PROPOSAL close all notifications with the same tag
+
 export function onNotificationClick (sw) {
   return (event) => {
     const url = event.notification.data?.url