fix(bulk-import): fixed Check All checkbox selection (#92)

* fix(bulk-import): fixed Check All checkbox selection

* fix(bulk-import): changing minor to patch

workspaces/bulk-import/.changeset/neat-news-rest.md

@@ -0,0 +1,5 @@
+---
+'@red-hat-developer-hub/backstage-plugin-bulk-import': patch
+---
+
+Fixed a bug in the bulk-import plugin where the "Check All" checkbox was incorrectly selected when the total number of selected repositories across multiple pages matched the number of rows in the current table

workspaces/bulk-import/plugins/bulk-import/src/components/AddRepositories/RepositoriesTable.tsx

@@ -34,6 +34,7 @@ import {
   evaluateRowForOrg,
   evaluateRowForRepo,
   filterSelectedForActiveDrawer,
+  filterSelectedRepositoriesOnActivePage,
   getComparator,
   updateWithNewSelectedRepositories,
 } from '../../utils/repository-utils';
@@ -250,7 +251,10 @@ export const RepositoriesTable = ({
     () => filterSelectedForActiveDrawer(tableData || [], selected),
     [tableData, selected],
   );
-
+  const selectedRepositoriesOnActivePage = React.useMemo(
+    () => filterSelectedRepositoriesOnActivePage(filteredData, selected),
+    [filteredData, selected],
+  );
   const getRowCount = () => {
     if (drawerOrganization) {
       return tableData?.filter(
@@ -296,7 +300,7 @@ export const RepositoriesTable = ({
             numSelected={
               drawerOrganization
                 ? Object.keys(selectedForActiveDrawer).length
-                : Object.keys(selected).length
+                : selectedRepositoriesOnActivePage.length
             }
             isDataLoading={loading}
             order={order}

workspaces/bulk-import/plugins/bulk-import/src/utils/repository-utils.tsx

@@ -202,6 +202,15 @@ export const filterSelectedForActiveDrawer = (
   );
 };
 
+export const filterSelectedRepositoriesOnActivePage = (
+  activePageTableData: AddRepositoryData[],
+  selectedRepositories: AddedRepositories,
+) => {
+  return Object.keys(selectedRepositories).filter(repoId => {
+    return activePageTableData.some(activeRepo => activeRepo.id === repoId);
+  });
+};
+
 export const urlHelper = (url: string) => {
   if (!url || url === '') {
     return '-';