Combine approve/deny modals into one modal

This commit is contained in:
mabashian
2020-11-13 10:10:40 -05:00
parent a9c3484387
commit aba14bfb8c

View File

@@ -129,11 +129,6 @@ function WorkflowApprovalsList({ i18n }) {
setSelected([]); setSelected([]);
}; };
const {
error: approveError,
dismissError: dismissApproveError,
} = useDismissableError(approveApprovalError);
const { const {
error: denyApprovalError, error: denyApprovalError,
isLoading: isDenyLoading, isLoading: isDenyLoading,
@@ -153,9 +148,9 @@ function WorkflowApprovalsList({ i18n }) {
}; };
const { const {
error: denyError, error: actionError,
dismissError: dismissDenyError, dismissError: dismissActionError,
} = useDismissableError(denyApprovalError); } = useDismissableError(approveApprovalError || denyApprovalError);
return ( return (
<> <>
@@ -248,26 +243,17 @@ function WorkflowApprovalsList({ i18n }) {
<ErrorDetail error={deletionError} /> <ErrorDetail error={deletionError} />
</AlertModal> </AlertModal>
)} )}
{approveError && ( {actionError && (
<AlertModal <AlertModal
isOpen={approveError} isOpen={actionError}
variant="error" variant="error"
title={i18n._(t`Error!`)} title={i18n._(t`Error!`)}
onClose={dismissApproveError} onClose={dismissActionError}
> >
{i18n._(t`Failed to approve one or more workflow approval.`)} {approveApprovalError
<ErrorDetail error={approveError} /> ? i18n._(t`Failed to approve one or more workflow approval.`)
</AlertModal> : i18n._(t`Failed to deny one or more workflow approval.`)}
)} <ErrorDetail error={actionError} />
{denyError && (
<AlertModal
isOpen={denyError}
variant="error"
title={i18n._(t`Error!`)}
onClose={dismissDenyError}
>
{i18n._(t`Failed to deny one or more workflow approval.`)}
<ErrorDetail error={denyError} />
</AlertModal> </AlertModal>
)} )}
</> </>