fix(bulk-import): disable deletion button for non-location sourced repos

Signed-off-by: Yi Cai <[email protected]>
redhat-developer · Nov 20, 2024 · fd188e2 · fd188e2
1 parent 201f00a
commit fd188e2
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 1 deletion.
diff --git a/workspaces/bulk-import/plugins/bulk-import/src/components/Repositories/DeleteRepository.tsx b/workspaces/bulk-import/plugins/bulk-import/src/components/Repositories/DeleteRepository.tsx
@@ -30,14 +30,20 @@ const DeleteRepository = ({ data }: { data: AddRepositoryData }) => {
     setOpenDialog(true);
   };
 
+  const tooltipMessage =
+    data.source === 'location'
+      ? 'Remove'
+      : 'This repository added to the app-config file. To remove it modify the file directly';
+
   return (
-    <Tooltip title="Remove">
+    <Tooltip title={tooltipMessage}>
       <span data-testid="delete-repository">
         <IconButton
           color="inherit"
           onClick={() => openDialog(data)}
           aria-label="Delete"
           size="large"
+          disabled={data.source !== 'location'}
         >
           <Delete />
         </IconButton>

diff --git a/workspaces/bulk-import/plugins/bulk-import/src/types/response-types.ts b/workspaces/bulk-import/plugins/bulk-import/src/types/response-types.ts
@@ -46,6 +46,7 @@ export type ImportJobStatus = {
   };
   status: string;
   id: string;
+  source?: 'location' | 'config' | 'other';
   lastUpdate: string;
   repository: Repository;
 };

diff --git a/workspaces/bulk-import/plugins/bulk-import/src/types/types.ts b/workspaces/bulk-import/plugins/bulk-import/src/types/types.ts
@@ -57,6 +57,7 @@ export type AddRepositoryData = {
   repoName?: string;
   orgName?: string;
   totalReposInOrg?: number;
+  source?: 'location' | 'config' | 'other';
   repoUrl?: string;
   organizationUrl?: string;
   selectedRepositories?: AddedRepositories;

diff --git a/workspaces/bulk-import/plugins/bulk-import/src/utils/repository-utils.tsx b/workspaces/bulk-import/plugins/bulk-import/src/utils/repository-utils.tsx
@@ -613,6 +613,7 @@ export const prepareDataForAddedRepositories = (
         ...acc,
         [id]: {
           id,
+          source: val.source,
           repoName: val.repository.name,
           defaultBranch: val.repository.defaultBranch,
           orgName: val.repository.organization,