Merge pull request #231 from GeekGene/fix/notifications-count-only-grows

fix(ui): cached notificationsCount should only grow over time
GeekGene · Sep 11, 2023 · a9bbb0f · a9bbb0f
2 parents 199cac1 + 014399b
commit a9bbb0f
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/ui/src/stores/notificationsRead.ts b/ui/src/stores/notificationsRead.ts
@@ -7,7 +7,6 @@ import { StateTree, defineStore } from "pinia";
 import { computed, ref } from "vue";
 import { Notification } from "@/types/types";
 import { encode, decode } from "@msgpack/msgpack";
-import { PersistedStateOptions } from "pinia-plugin-persistedstate";
 
 const notificationToKey = (notification: Notification) => {
   const keyObj = {
@@ -53,7 +52,9 @@ export const makeUseNotificationsReadStore = (client: AppAgentClient) =>
       }
 
       function setNotificationsCount(count: number) {
-        notificationsCount.value = count;
+        if (count > notificationsCount.value) {
+          notificationsCount.value = count;
+        }
       }
 
       return {