add mass delete time entries frontend, closes ST-450

solidtime-io · Oct 7, 2024 · bfbcd3a · bfbcd3a
1 parent e0f218e
commit bfbcd3a
Show file tree

Hide file tree

Showing 4 changed files with 72 additions and 4 deletions.
diff --git a/openapi.json.client.ts b/openapi.json.client.ts
@@ -2203,6 +2203,54 @@ Users with the permission &#x60;time-entries:view:own&#x60; can only use this en
             },
         ],
     },
+    {
+        method: 'delete',
+        path: '/v1/organizations/:organization/time-entries',
+        alias: 'deleteTimeEntries',
+        requestFormat: 'json',
+        parameters: [
+            {
+                name: 'organization',
+                type: 'Path',
+                schema: z.string(),
+            },
+            {
+                name: 'ids',
+                type: 'Query',
+                schema: z.array(z.string().uuid()),
+            },
+        ],
+        response: z
+            .object({ success: z.string(), error: z.string() })
+            .passthrough(),
+        errors: [
+            {
+                status: 401,
+                description: `Unauthenticated`,
+                schema: z.object({ message: z.string() }).passthrough(),
+            },
+            {
+                status: 403,
+                description: `Authorization error`,
+                schema: z.object({ message: z.string() }).passthrough(),
+            },
+            {
+                status: 404,
+                description: `Not found`,
+                schema: z.object({ message: z.string() }).passthrough(),
+            },
+            {
+                status: 422,
+                description: `Validation error`,
+                schema: z
+                    .object({
+                        message: z.string(),
+                        errors: z.record(z.array(z.string())),
+                    })
+                    .passthrough(),
+            },
+        ],
+    },
     {
         method: 'put',
         path: '/v1/organizations/:organization/time-entries/:timeEntry',

diff --git a/resources/js/Components/Common/TimeEntry/TimeEntryAggregateRow.vue b/resources/js/Components/Common/TimeEntry/TimeEntryAggregateRow.vue
@@ -137,7 +137,7 @@ const expanded = ref(false);
                         class="opacity-20 hidden sm:flex group-hover:opacity-100"></TimeTrackerStartStop>
                     <TimeEntryMoreOptionsDropdown
                         @delete="
-                            deleteTimeEntries([timeEntry])
+                            deleteTimeEntries(timeEntry?.timeEntries ?? [])
                         "></TimeEntryMoreOptionsDropdown>
                 </div>
             </div>

diff --git a/resources/js/Pages/Time.vue b/resources/js/Pages/Time.vue
@@ -58,9 +58,7 @@ async function startTimeEntry(
 }
 
 function deleteTimeEntries(timeEntries: TimeEntry[]) {
-    timeEntries.forEach((entry) => {
-        useTimeEntriesStore().deleteTimeEntry(entry.id);
-    });
+    useTimeEntriesStore().deleteTimeEntries(timeEntries);
     fetchTimeEntries();
 }
 

diff --git a/resources/js/utils/useTimeEntries.ts b/resources/js/utils/useTimeEntries.ts
@@ -135,6 +135,27 @@ export const useTimeEntriesStore = defineStore('timeEntries', () => {
         }
     }
 
+    async function deleteTimeEntries(timeEntries: TimeEntry[]) {
+        const organizationId = getCurrentOrganizationId();
+        const timeEntryIds = timeEntries.map((entry) => entry.id);
+        if (organizationId) {
+            await handleApiRequestNotifications(
+                () =>
+                    api.deleteTimeEntries(undefined, {
+                        queries: {
+                            ids: timeEntryIds,
+                        },
+                        params: {
+                            organization: organizationId,
+                        },
+                    }),
+                'Time entries deleted successfully',
+                'Failed to delete time entries'
+            );
+            await fetchTimeEntries();
+        }
+    }
+
     return {
         timeEntries,
         fetchTimeEntries,
@@ -143,5 +164,6 @@ export const useTimeEntriesStore = defineStore('timeEntries', () => {
         deleteTimeEntry,
         fetchMoreTimeEntries,
         allTimeEntriesLoaded,
+        deleteTimeEntries,
     };
 });