Correct batch delete targets

Author: sunnylqm

src/pages/audit-logs.tsx

@@ -94,12 +94,13 @@ const actionMap: Record<string, string> = {
   'POST /upload': '上传文件',
   'POST /app/{id}/package/create': '创建原生包',
   'PUT /app/{id}/package/{id}': '修改原生包设置',
+  'DELETE /app/{id}/package': '批量删除原生包',
   'DELETE /app/{id}/package/{id}': '删除原生包',
   'POST /app/{id}/version/create': '创建热更包',
   'PUT /app/{id}/version/{id}': '修改热更包设置',
+  'DELETE /app/{id}/version': '批量删除热更包',
   'DELETE /app/{id}/version/{id}': '删除热更包',
   'POST /app/{id}/binding': '创建/更新绑定',
-  'DELETE /app/{id}/binding': '批量删除绑定',
   'DELETE /app/{id}/binding/{id}': '删除绑定',
   'POST /api-token/create': '创建 API Key',
   'DELETE /api-token/{id}': '删除 API Key',

src/pages/manage/components/version-table.tsx

@@ -120,9 +120,7 @@ function removeSelectedVersions({
     maskClosable: true,
     okButtonProps: { danger: true },
     async onOk() {
-      await Promise.all(
-        selected.map((id) => api.deleteVersion({ appId, versionId: id })),
-      );
+      await api.deleteVersions({ appId, versionIds: selected });
     },
   });
 }

src/services/api.ts

@@ -129,6 +129,22 @@ export const api = {
         }),
       );
     }),
+  deletePackages: ({
+    appId,
+    packageIds,
+  }: {
+    appId: number;
+    packageIds: number[];
+  }) =>
+    request('delete', `/app/${appId}/package`, { packageIds }).then(() => {
+      const packageIdSet = new Set(packageIds);
+      queryClient.setQueryData(
+        ['packages', appId],
+        ({ data }: { data: Package[] }) => ({
+          data: data?.filter((i) => !packageIdSet.has(i.id)),
+        }),
+      );
+    }),
   // version
   getVersions: ({
     appId,
@@ -180,6 +196,33 @@ export const api = {
             : undefined,
       );
     }),
+  deleteVersions: ({
+    appId,
+    versionIds,
+  }: {
+    appId: number;
+    versionIds: number[];
+  }) =>
+    request<{ count: number }>('delete', `/app/${appId}/version`, {
+      versionIds,
+    }).then((response) => {
+      const versionIdSet = new Set(versionIds);
+      const deletedCount = response?.count ?? versionIds.length;
+      queryClient.setQueriesData(
+        { queryKey: versionKeys.byApp(appId) },
+        (old?: { data: Version[]; count?: number }) =>
+          old
+            ? {
+                ...old,
+                data: old.data?.filter((i) => !versionIdSet.has(i.id)),
+                count: Math.max(
+                  (old.count ?? old.data.length) - deletedCount,
+                  0,
+                ),
+              }
+            : undefined,
+      );
+    }),
   // order
   createOrder: (params: { tier?: string }) =>
     request<{ payUrl: string }>('post', '/orders', params),
@@ -221,26 +264,6 @@ export const api = {
             : undefined,
       );
     }),
-  deleteBindings: ({
-    appId,
-    bindingIds,
-  }: {
-    appId: number;
-    bindingIds: number[];
-  }) =>
-    request('delete', `/app/${appId}/binding`, { bindingIds }).then(() => {
-      const bindingIdSet = new Set(bindingIds);
-      queryClient.setQueriesData(
-        { queryKey: ['bindings', appId] },
-        (old?: { data: Binding[] }) =>
-          old
-            ? {
-                ...old,
-                data: old.data?.filter((i) => !bindingIdSet.has(i.id)),
-              }
-            : undefined,
-      );
-    }),
   // audit logs
   getAuditLogs: ({
     offset = 0,