fix: api key delete bug (#1524)

packages/web/hooks/useConfirm.tsx

@@ -41,23 +41,23 @@ export const useConfirm = (props?: {
 
   const { isOpen, onOpen, onClose } = useDisclosure();
 
-  const confirmCb = useRef<any>();
+  const confirmCb = useRef<Function>();
   const cancelCb = useRef<any>();
 
-  return {
+  const openConfirm = (
-    openConfirm: useCallback(
+    confirm?: Function,
-      (confirm?: any, cancel?: any, customContent?: string | React.ReactNode) => {
+    cancel?: any,
+    customContent?: string | React.ReactNode
+  ) => {
     confirmCb.current = confirm;
     cancelCb.current = cancel;
 
     customContent && setCustomContent(customContent);
 
     return onOpen;
-      },
+  };
-      [onOpen]
+
-    ),
+  const ConfirmModal = useCallback(
-    onClose,
-    ConfirmModal: useCallback(
     ({
       closeText = t('common.Cancel'),
       confirmText = t('common.Confirm'),
@@ -122,6 +122,11 @@ export const useConfirm = (props?: {
       );
     },
     [customContent, hideFooter, iconSrc, isOpen, map.bg, onClose, showCancel, t, title]
-    )
+  );
+
+  return {
+    openConfirm,
+    onClose,
+    ConfirmModal
   };
 };

projects/app/src/components/support/apikey/Table.tsx

@@ -68,7 +68,9 @@ const ApiKeyTable = ({ tips, appId }: { tips: string; appId?: string }) => {
   });
 
   const { mutate: onclickRemove, isLoading: isDeleting } = useMutation({
-    mutationFn: async (id: string) => delOpenApiById(id),
+    mutationFn: async (id: string) => {
+      return delOpenApiById(id);
+    },
     onSuccess() {
       refetch();
     }
@@ -212,7 +214,7 @@ const ApiKeyTable = ({ tips, appId }: { tips: string; appId?: string }) => {
                         label: t('common.Delete'),
                         icon: 'delete',
                         type: 'danger',
-                        onClick: openConfirm(() => onclickRemove(_id))
+                        onClick: () => openConfirm(() => onclickRemove(_id))()
                       }
                     ]}
                   />