Merge pull request #37 from carlosthe19916/fix/certificate-unique-id-definition

fix: include index on certificate unique identifier

Author: abhinandan13jan

client/src/app/hooks/query-utils.ts

@@ -16,16 +16,16 @@ export const useWithUiId = <T>(
   /** Source data to modify. */
   data: T[] | undefined,
   /** Generate the unique id for a specific `T`. */
-  generator: (item: T) => string
+  generator: (item: T, index: number) => string
 ): WithUiId<T>[] => {
   const result = useMemo(() => {
     if (!data || data.length === 0) {
       return [];
     }
 
-    const dataWithUiId: WithUiId<T>[] = data.map((item) => ({
+    const dataWithUiId: WithUiId<T>[] = data.map((item, index) => ({
       ...item,
-      [UI_UNIQUE_ID]: generator(item),
+      [UI_UNIQUE_ID]: generator(item, index),
     }));
 
     return dataWithUiId;

client/src/app/pages/TrustRoot/components/Certificates.tsx

@@ -35,7 +35,10 @@ interface ICertificatesTableProps {
 }
 
 export const CertificatesTable: React.FC<ICertificatesTableProps> = ({ certificates, isFetching, fetchError }) => {
-  const items = useWithUiId(certificates, (item) => `${item.type}-${item.issuer}-${item.subject}-${item.target}`);
+  const items = useWithUiId(
+    certificates,
+    (item, index) => `${index}-${item.type}-${item.issuer}-${item.subject}-${item.target}`
+  );
 
   const tableState = usePFToolbarTable({
     items,