1611 comments

Author: jessicayzhao

src/frontend/src/apis/finance.api.ts

@@ -162,7 +162,7 @@ export const approveReimbursementRequest = (id: string) => {
  * Deny Reimbursement Request
  *
  * @param id of the reimbursement request being denied by finance
- * @returns the reimbursement request with the given id
+ * @returns the denied reimbursement status
  */
 export const denyReimbursementRequest = (id: string) => {
   return axios.post(apiUrls.financeDenyReimbursementRequest(id));

src/frontend/src/hooks/finance.hooks.ts

@@ -265,7 +265,7 @@ export const useApproveReimbursementRequest = (id: string) => {
  * Custom react hook to deny a reimbursement request for the finance team
  *
  * @param id id of the reimbursement request to deny
- * @returns the denied reimbursement request
+ * @returns the denied reimbursement request status
  */
 export const useDenyReimbursementRequest = (id: string) => {
   const queryClient = useQueryClient();

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

@@ -80,6 +80,7 @@ const ReimbursementRequestDetailsView: React.FC<ReimbursementRequestDetailsViewP
   const handleDeny = () => {
     try {
       denyReimbursementRequest();
+      setShowDenyModal(false);
     } catch (e: unknown) {
       if (e instanceof Error) {
         toast.error(e.message, 3000);