Skip to content

Commit 2ce8b82

Browse files
committed
#1611 approve/denied disabled if denied
1 parent c81262f commit 2ce8b82

2 files changed

Lines changed: 8 additions & 1 deletion

File tree

src/backend/src/services/reimbursement-requests.services.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -684,6 +684,10 @@ export default class ReimbursementRequestService {
684684
throw new HttpException(400, 'This reimbursement request has already been approved');
685685
}
686686

687+
if (reimbursementRequest.reimbursementStatuses.some((status) => status.type === ReimbursementStatusType.DENIED)) {
688+
throw new HttpException(400, 'This reimbursement request has already been denied');
689+
}
690+
687691
const reimbursementStatus = await prisma.reimbursement_Status.create({
688692
data: {
689693
type: ReimbursementStatusType.SABO_SUBMITTED,

src/frontend/src/pages/FinancePage/ReimbursementRequestDetailPage/ReimbursementRequestDetailsView.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,10 @@ const ReimbursementRequestDetailsView: React.FC<ReimbursementRequestDetailsViewP
253253
title: 'Approve',
254254
onClick: () => setShowSubmitToSaboModal(true),
255255
icon: <CheckIcon />,
256-
disabled: !user.isFinance || isReimbursementRequestSaboSubmitted(reimbursementRequest)
256+
disabled:
257+
!user.isFinance ||
258+
isReimbursementRequestSaboSubmitted(reimbursementRequest) ||
259+
isReimbursementRequestDenied(reimbursementRequest)
257260
},
258261
{
259262
title: 'Deny',

0 commit comments

Comments
 (0)