Fix handleOpen

PeculiarVentures · Jun 24, 2024 · d189624 · d189624
1 parent 58213a1
commit d189624
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/src/components/certificates-list/CertificatesList.tsx b/src/components/certificates-list/CertificatesList.tsx
@@ -28,7 +28,11 @@ import styles from "./styles/index.module.scss";
 
 interface CertificatesListProps {
   certificates: CertificateProps[];
-  onDelete: (certificateId: string, providerId: string, label: string) => void;
+  onDelete: (params: {
+    certificateIndex: string;
+    providerId: string;
+    label: string;
+  }) => void;
   onViewDetails: (certificate: CertificateProps) => void;
 }
 
@@ -139,7 +143,11 @@ export const CertificatesList: React.FunctionComponent<
                       tabIndex={0}
                       title={t("certificates.list.action.delete")}
                       onClick={() =>
-                        onDelete(index, providerID, label as string)
+                        onDelete({
+                          certificateIndex: index,
+                          providerId: providerID,
+                          label: label as string,
+                        })
                       }
                       size="small"
                       className={styles.action_icon_button}

diff --git a/src/dialogs/certificate-delete-dialog/useCertificateDeleteDialog.tsx b/src/dialogs/certificate-delete-dialog/useCertificateDeleteDialog.tsx
@@ -27,16 +27,8 @@ export function useCertificateDeleteDialog(
   const [isOpen, setIsOpen] = React.useState(false);
   const openParamsRef = React.useRef<UseCertificateDeleteDialogOpenParams>();
 
-  const handleOpen = (
-    certificateIndex: string,
-    providerId: string,
-    label: string
-  ) => {
-    openParamsRef.current = {
-      certificateIndex,
-      providerId,
-      label,
-    };
+  const handleOpen = (params: UseCertificateDeleteDialogOpenParams) => {
+    openParamsRef.current = params;
     setIsOpen(true);
   };