Merge pull request #37595 from ShridharGoel/notif-tags

Show only one update available notification at a time
Expensify · Mar 11, 2024 · d21bda0 · d21bda0
2 parents fda1b3a + 8dd0a39
commit d21bda0
Showing 1 changed file with 21 additions and 4 deletions.
diff --git a/src/libs/Notification/LocalNotification/BrowserNotifications.ts b/src/libs/Notification/LocalNotification/BrowserNotifications.ts
@@ -44,7 +44,15 @@ function canUseBrowserNotifications(): Promise<boolean> {
  * @param icon Path to icon
  * @param data extra data to attach to the notification
  */
-function push(title: string, body = '', icon: string | ImageSourcePropType = '', data: LocalNotificationData = {}, onClick: LocalNotificationClickHandler = () => {}, silent = false) {
+function push(
+    title: string,
+    body = '',
+    icon: string | ImageSourcePropType = '',
+    data: LocalNotificationData = {},
+    onClick: LocalNotificationClickHandler = () => {},
+    silent = false,
+    tag = '',
+) {
     canUseBrowserNotifications().then((canUseNotifications) => {
         if (!canUseNotifications) {
             return;
@@ -57,6 +65,7 @@ function push(title: string, body = '', icon: string | ImageSourcePropType = '',
             icon: String(icon),
             data,
             silent,
+            tag,
         });
         notificationCache[notificationID].onclick = () => {
             onClick();
@@ -124,9 +133,17 @@ export default {
      * Create a notification to indicate that an update is available.
      */
     pushUpdateAvailableNotification() {
-        push('Update available', 'A new version of this app is available!', '', {}, () => {
-            AppUpdate.triggerUpdateAvailable();
-        });
+        push(
+            'Update available',
+            'A new version of this app is available!',
+            '',
+            {},
+            () => {
+                AppUpdate.triggerUpdateAvailable();
+            },
+            false,
+            'UpdateAvailable',
+        );
     },
 
     /**