Skip to content

Commit

Permalink
fix: hide recover button for successful executions (#203)
Browse files Browse the repository at this point in the history
Signed-off-by: Pianist038801 <[email protected]>

Co-authored-by: Pianist038801 <[email protected]>
  • Loading branch information
Pianist038801 and Pianist038801 authored Sep 16, 2021
1 parent ad9f889 commit 81dd608
Showing 1 changed file with 27 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import * as React from 'react';
import { Link as RouterLink } from 'react-router-dom';
import { history } from 'routes/history';
import { Routes } from 'routes/routes';
import { WorkflowExecutionPhase } from 'models/Execution/enums';
import { ExecutionInputsOutputsModal } from '../ExecutionInputsOutputsModal';
import { ExecutionStatusBadge } from '../ExecutionStatusBadge';
import { TerminateExecutionButton } from '../TerminateExecution/TerminateExecutionButton';
Expand Down Expand Up @@ -123,24 +124,36 @@ export const ExecutionDetailsAppBarContent: React.FC<{
await recoverExecution();
}, [recoverExecution]);

const isRecoverVisible = React.useMemo(
() =>
[
WorkflowExecutionPhase.FAILED,
WorkflowExecutionPhase.ABORTED,
WorkflowExecutionPhase.TIMED_OUT
].includes(phase),
[phase]
);

const actionContent = isRunning ? (
<TerminateExecutionButton className={styles.actionButton} />
) : isTerminal ? (
<>
<Button
variant="outlined"
color="primary"
disabled={recovering}
className={classnames(
styles.actionButton,
commonStyles.buttonWhiteOutlined
)}
onClick={onClickRecover}
size="small"
>
Recover
{recovering && <ButtonCircularProgress />}
</Button>
{isRecoverVisible && (
<Button
variant="outlined"
color="primary"
disabled={recovering}
className={classnames(
styles.actionButton,
commonStyles.buttonWhiteOutlined
)}
onClick={onClickRecover}
size="small"
>
Recover
{recovering && <ButtonCircularProgress />}
</Button>
)}
<Button
variant="outlined"
color="primary"
Expand Down

0 comments on commit 81dd608

Please sign in to comment.