From e0fb65ab47f214416ba355eedaacdfa79ebef6a2 Mon Sep 17 00:00:00 2001
From: Kristina <kristin.fefelova@gmail.com>
Date: Tue, 5 Mar 2024 17:02:52 +0400
Subject: [PATCH] Add confirmation modal for archive all and add read all
 option (#4880)

Signed-off-by: Kristina Fefelova <kristin.fefelova@gmail.com>
---
 .../src/components/ButtonWithDropdown.svelte  |  5 +-
 plugins/notification-assets/lang/en.json      |  5 +-
 plugins/notification-assets/lang/ru.json      |  5 +-
 .../src/components/inbox/Inbox.svelte         | 82 +++++++++++++++----
 .../inbox/InboxNotificationPresenter.svelte   |  2 +-
 .../src/inboxNotificationsClient.ts           | 17 ++++
 plugins/notification/src/index.ts             |  6 +-
 7 files changed, 99 insertions(+), 23 deletions(-)

diff --git a/packages/ui/src/components/ButtonWithDropdown.svelte b/packages/ui/src/components/ButtonWithDropdown.svelte
index e6406befe9..15b7116fac 100644
--- a/packages/ui/src/components/ButtonWithDropdown.svelte
+++ b/packages/ui/src/components/ButtonWithDropdown.svelte
@@ -60,7 +60,7 @@
       id={mainButtonId}
     >
       <div class="flex w-full" slot="content">
-        <div class="flex-row-center w-full flex-between">
+        <div class="flex-row-center w-full flex-between relative">
           {#if label}
             <Label {label} />
             <slot name="content" />
@@ -81,10 +81,11 @@
 
 <style lang="scss">
   .vertical-divider {
+    position: absolute;
     background-color: var(--theme-content-color);
     min-width: 1px;
     opacity: 0.25;
-    margin-right: -0.75rem;
+    right: -0.75rem;
 
     &.primary,
     &.secondary,
diff --git a/plugins/notification-assets/lang/en.json b/plugins/notification-assets/lang/en.json
index 1da774f82e..c7a43dcc84 100644
--- a/plugins/notification-assets/lang/en.json
+++ b/plugins/notification-assets/lang/en.json
@@ -36,6 +36,9 @@
     "Message": "Message",
     "FlatList": "Flat list",
     "GroupedList": "Grouped list",
-    "ArchiveAll": "Archive all"
+    "ArchiveAll": "Archive all",
+    "ReadAll": "Read all",
+    "ArchiveAllConfirmationTitle": "Archive all notifications?",
+    "ArchiveAllConfirmationMessage": "Are you sure you want to archive all notifications? This operation cannot be undone."
   }
 }
diff --git a/plugins/notification-assets/lang/ru.json b/plugins/notification-assets/lang/ru.json
index e2512a42cf..1427faa418 100644
--- a/plugins/notification-assets/lang/ru.json
+++ b/plugins/notification-assets/lang/ru.json
@@ -36,6 +36,9 @@
     "Message": "Сообщение",
     "FlatList": "Flat list",
     "GroupedList": "Grouped list",
-    "ArchiveAll": "Архивировать все"
+    "ArchiveAll": "Архивировать все",
+    "ReadAll": "Прочитать все",
+    "ArchiveAllConfirmationTitle": "Архивировать все уведомления?",
+    "ArchiveAllConfirmationMessage": "Вы уверены, что хотите заархивировать все уведомления? Эту операцию невозможно отменить."
   }
 }
diff --git a/plugins/notification-resources/src/components/inbox/Inbox.svelte b/plugins/notification-resources/src/components/inbox/Inbox.svelte
index b8dbdf307a..bc698c6ae8 100644
--- a/plugins/notification-resources/src/components/inbox/Inbox.svelte
+++ b/plugins/notification-resources/src/components/inbox/Inbox.svelte
@@ -18,7 +18,7 @@
     DisplayInboxNotification,
     DocNotifyContext
   } from '@hcengineering/notification'
-  import { ActionContext, createQuery, getClient } from '@hcengineering/presentation'
+  import { ActionContext, createQuery, getClient, MessageBox } from '@hcengineering/presentation'
   import view, { Viewlet } from '@hcengineering/view'
   import {
     AnyComponent,
@@ -32,7 +32,10 @@
     TabItem,
     TabList,
     Location,
-    ModernButton
+    IconDropdown,
+    ButtonWithDropdown,
+    IconCheckAll,
+    showPopup
   } from '@hcengineering/ui'
   import chunter, { ThreadMessage } from '@hcengineering/chunter'
   import { Ref, WithLookup } from '@hcengineering/core'
@@ -92,11 +95,11 @@
   let viewlet: WithLookup<Viewlet> | undefined
   let loading = true
 
-  client.findAll(notification.class.ActivityNotificationViewlet, {}).then((res) => {
+  void client.findAll(notification.class.ActivityNotificationViewlet, {}).then((res) => {
     viewlets = res
   })
 
-  $: getDisplayInboxNotifications($notificationsByContextStore, filter).then((res) => {
+  $: void getDisplayInboxNotifications($notificationsByContextStore, filter).then((res) => {
     displayNotifications = res
   })
   $: displayContextsIds = new Set(displayNotifications.map(({ docNotifyContext }) => docNotifyContext))
@@ -104,7 +107,7 @@
   $: filteredNotifications = filterNotifications(selectedTabId, displayNotifications, $notifyContextsStore)
 
   locationStore.subscribe((newLocation) => {
-    syncLocation(newLocation)
+    void syncLocation(newLocation)
   })
 
   inboxClient.activityInboxNotifications.subscribe((notifications) => {
@@ -121,7 +124,7 @@
     }
   )
 
-  async function syncLocation (newLocation: Location) {
+  async function syncLocation (newLocation: Location): Promise<void> {
     const loc = await resolveLocation(newLocation)
 
     selectedContextId = loc?.loc.path[3] as Ref<DocNotifyContext> | undefined
@@ -165,13 +168,13 @@
     )
   }
 
-  function selectTab (event: CustomEvent) {
+  function selectTab (event: CustomEvent): void {
     if (event.detail !== undefined) {
       selectedTabId = event.detail.id
     }
   }
 
-  async function selectContext (event?: CustomEvent) {
+  async function selectContext (event?: CustomEvent): Promise<void> {
     selectedContext = event?.detail?.context
     selectedContextId = selectedContext?._id
 
@@ -202,7 +205,7 @@
     }
   }
 
-  async function updateSelectedPanel (selectedContext?: DocNotifyContext) {
+  async function updateSelectedPanel (selectedContext?: DocNotifyContext): Promise<void> {
     if (selectedContext === undefined) {
       selectedComponent = undefined
       return
@@ -236,7 +239,7 @@
     selectedTabId: string,
     displayNotifications: DisplayInboxNotification[],
     notifyContexts: DocNotifyContext[]
-  ) {
+  ): DisplayInboxNotification[] {
     if (selectedTabId === allTab.id) {
       return displayNotifications
     }
@@ -258,7 +261,35 @@
   })
 
   function archiveAll (): void {
-    void inboxClient.deleteAllNotifications()
+    showPopup(
+      MessageBox,
+      {
+        label: notification.string.ArchiveAllConfirmationTitle,
+        message: notification.string.ArchiveAllConfirmationMessage
+      },
+      'top',
+      (result?: boolean) => {
+        if (result === true) {
+          void inboxClient.deleteAllNotifications()
+        }
+      }
+    )
+  }
+
+  function readAll (): void {
+    void inboxClient.readAllNotifications()
+  }
+
+  async function dropdownItemSelected (id: 'archive' | 'read'): Promise<void> {
+    if (id == null) return
+
+    if (id === 'archive') {
+      archiveAll()
+    }
+
+    if (id === 'read') {
+      readAll()
+    }
   }
 </script>
 
@@ -288,12 +319,29 @@
             />
           </div>
           <div class="flex flex-gap-2">
-            <ModernButton
-              label={notification.string.ArchiveAll}
-              icon={view.icon.Archive}
-              size="small"
-              on:click={archiveAll}
-            />
+            {#if displayNotifications.length > 0}
+              <ButtonWithDropdown
+                justify="left"
+                kind="regular"
+                label={notification.string.ReadAll}
+                icon={IconCheckAll}
+                on:click={readAll}
+                dropdownItems={[
+                  {
+                    id: 'read',
+                    icon: IconCheckAll,
+                    label: notification.string.ReadAll
+                  },
+                  {
+                    id: 'archive',
+                    icon: view.icon.Archive,
+                    label: notification.string.ArchiveAll
+                  }
+                ]}
+                dropdownIcon={IconDropdown}
+                on:dropdown-selected={(ev) => dropdownItemSelected(ev.detail)}
+              />
+            {/if}
             <Filter bind:filter />
           </div>
         </div>
diff --git a/plugins/notification-resources/src/components/inbox/InboxNotificationPresenter.svelte b/plugins/notification-resources/src/components/inbox/InboxNotificationPresenter.svelte
index af0b2879ab..a26bd649c7 100644
--- a/plugins/notification-resources/src/components/inbox/InboxNotificationPresenter.svelte
+++ b/plugins/notification-resources/src/components/inbox/InboxNotificationPresenter.svelte
@@ -17,7 +17,7 @@
   import { getClient } from '@hcengineering/presentation'
   import { Component } from '@hcengineering/ui'
   import { Class, Doc, Ref } from '@hcengineering/core'
-  import { ActivityNotificationViewlet, DisplayInboxNotification, DocNotifyContext } from '@hcengineering/notification'
+  import { ActivityNotificationViewlet, DisplayInboxNotification } from '@hcengineering/notification'
 
   export let value: DisplayInboxNotification
   export let embedded = false
diff --git a/plugins/notification-resources/src/inboxNotificationsClient.ts b/plugins/notification-resources/src/inboxNotificationsClient.ts
index cddce3e17b..5314e5a0af 100644
--- a/plugins/notification-resources/src/inboxNotificationsClient.ts
+++ b/plugins/notification-resources/src/inboxNotificationsClient.ts
@@ -266,4 +266,21 @@ export class InboxNotificationsClientImpl implements InboxNotificationsClient {
       await doneOp()
     }
   }
+
+  async readAllNotifications (): Promise<void> {
+    const doneOp = await getClient().measure('readAllNotifications')
+    const ops = getClient().apply(generateId())
+
+    try {
+      const inboxNotifications = get(this.inboxNotifications) ?? []
+      for (const notification of inboxNotifications) {
+        if (!notification.isViewed) {
+          await ops.update(notification, { isViewed: true })
+        }
+      }
+    } finally {
+      await ops.commit()
+      await doneOp()
+    }
+  }
 }
diff --git a/plugins/notification/src/index.ts b/plugins/notification/src/index.ts
index 0b1d098fc7..c2a002e629 100644
--- a/plugins/notification/src/index.ts
+++ b/plugins/notification/src/index.ts
@@ -283,6 +283,7 @@ export interface InboxNotificationsClient {
   unreadNotifications: (client: TxOperations, ids: Array<Ref<InboxNotification>>) => Promise<void>
   deleteNotifications: (client: TxOperations, ids: Array<Ref<InboxNotification>>) => Promise<void>
   deleteAllNotifications: () => Promise<void>
+  readAllNotifications: () => Promise<void>
 }
 
 /**
@@ -390,7 +391,10 @@ const notification = plugin(notificationId, {
     FlatList: '' as IntlString,
     GroupedList: '' as IntlString,
     All: '' as IntlString,
-    ArchiveAll: '' as IntlString
+    ArchiveAll: '' as IntlString,
+    ReadAll: '' as IntlString,
+    ArchiveAllConfirmationTitle: '' as IntlString,
+    ArchiveAllConfirmationMessage: '' as IntlString
   },
   function: {
     GetInboxNotificationsClient: '' as Resource<InboxNotificationsClientFactory>,