refactor: edit button to delete

caraml-dev · May 12, 2023 · c1a6458 · c1a6458
1 parent aab65c6
commit c1a6458
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/ui/src/jobs/components/modal/DeleteJobModal.js b/ui/src/jobs/components/modal/DeleteJobModal.js
@@ -46,7 +46,7 @@ export const DeleteJobModal = ({
           You are about to terminate Ensembling Jobs <b>{job.name}</b>
         </p>
       }
-      confirmButtonText="Terminate"
+      confirmButtonText={["failed", "failed_submission", "failed_building", "completed"].includes(job.status) ? 'Delete' : 'Terminate'}
       confirmButtonColor="danger">
       {(onSubmit) =>
         (deleteJobRef.current = openModal(onSubmit)) &&

diff --git a/ui/src/jobs/list/ListEnsemblingJobsTable.js b/ui/src/jobs/list/ListEnsemblingJobsTable.js
@@ -158,10 +158,10 @@ export const ListEnsemblingJobsTable = ({
               <EuiButtonEmpty
                   onClick={() => handleDeleteJob(item)}
                   color={"danger"}
-                  iconType="minusInCircle"
+                  iconType={["failed", "failed_submission", "failed_building", "completed"].includes(item.status) ? 'trash' : 'minusInCircle' }
                   iconSide="left"
                   size="xs">
-                <EuiText size="xs" >Terminate</EuiText>
+                <EuiText size="xs"> {["failed", "failed_submission", "failed_building", "completed"].includes(item.status) ? 'Delete' : 'Terminate' } </EuiText>
               </EuiButtonEmpty>
             </EuiFlexItem>
           </EuiFlexItem>