#1568 - all finance members have all perms

Author: RChandler234

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

@@ -24,7 +24,6 @@ import {
   removeDeletedReceiptPictures,
   updateReimbursementProducts,
   validateReimbursementProducts,
-  validateUserIsHeadOfFinanceTeam,
   validateUserIsPartOfFinanceTeam
 } from '../utils/reimbursement-requests.utils';
 import {
@@ -334,7 +333,7 @@ export default class ReimbursementRequestService {
    * @returns reimbursement requests with no advisor approved reimbursement status
    */
   static async getPendingAdvisorList(requester: User): Promise<ReimbursementRequest[]> {
-    await validateUserIsHeadOfFinanceTeam(requester);
+    await validateUserIsPartOfFinanceTeam(requester);
 
     const requestsPendingAdvisors = await prisma.reimbursement_Request.findMany({
       where: {
@@ -360,7 +359,7 @@ export default class ReimbursementRequestService {
    * @param saboNumbers the sabo numbers of the reimbursement requests to send
    */
   static async sendPendingAdvisorList(sender: User, saboNumbers: number[]) {
-    await validateUserIsHeadOfFinanceTeam(sender);
+    await validateUserIsPartOfFinanceTeam(sender);
 
     if (saboNumbers.length === 0) throw new HttpException(400, 'Need to send at least one Sabo #!');
 

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

@@ -233,15 +233,3 @@ export const isAuthUserHeadOfFinance = (user: Prisma.UserGetPayload<typeof authU
 const isTeamIdInList = (teamId: string, teamsList: Team[]) => {
   return teamsList.map((team) => team.teamId).includes(teamId);
 };
-
-export const validateUserIsHeadOfFinanceTeam = async (user: User) => {
-  const financeTeam = await prisma.team.findUnique({
-    where: { teamId: process.env.FINANCE_TEAM_ID }
-  });
-
-  if (!financeTeam) throw new HttpException(500, 'Finance team does not exist!');
-
-  if (!(financeTeam.headId === user.userId)) {
-    throw new AccessDeniedException('You are not the head of the finance team!');
-  }
-};

src/frontend/src/pages/FinancePage/FinancePage.tsx

@@ -51,7 +51,7 @@ const FinancePage = () => {
     error: allPendingAdvisorListError
   } = useGetPendingAdvisorList();
 
-  const { isFinance, isHeadOfFinance } = user;
+  const { isFinance } = user;
 
   const [anchorEl, setAnchorEl] = useState<null | HTMLElement>(null);
 
@@ -60,16 +60,16 @@ const FinancePage = () => {
 
   if (isFinance && allReimbursementRequestsIsError) return <ErrorPage message={allReimbursementRequestsError?.message} />;
   if (userReimbursementRequestIsError) return <ErrorPage message={userReimbursementRequestError?.message} />;
-  if (isHeadOfFinance && allPendingAdvisorListIsError) return <ErrorPage message={allPendingAdvisorListError?.message} />;
+  if (isFinance && allPendingAdvisorListIsError) return <ErrorPage message={allPendingAdvisorListError?.message} />;
   if (
     (isFinance && (allReimbursementRequestsIsLoading || !allReimbursementRequests)) ||
     userReimbursementRequestIsLoading ||
     !userReimbursementRequests ||
-    (isHeadOfFinance && !allPendingAdvisorList)
+    (isFinance && !allPendingAdvisorList)
   )
     return <LoadingIndicator />;
 
-  if (isHeadOfFinance && (!allPendingAdvisorList || allPendingAdvisorListIsLoading)) return <LoadingIndicator />;
+  if (isFinance && (!allPendingAdvisorList || allPendingAdvisorListIsLoading)) return <LoadingIndicator />;
 
   const handleClick = (event: React.MouseEvent<HTMLElement>) => {
     setAnchorEl(event.currentTarget);
@@ -112,7 +112,7 @@ const FinancePage = () => {
             handleDropdownClose();
             setShowPendingAdvisorListModal(true);
           }}
-          disabled={!isHeadOfFinance}
+          disabled={!isFinance}
         >
           <ListItemIcon>
             <ListAltIcon fontSize="small" />
@@ -131,7 +131,7 @@ const FinancePage = () => {
 
   return (
     <PageLayout title="Finance" headerRight={financeActionsDropdown}>
-      {isHeadOfFinance && (
+      {isFinance && (
         <PendingAdvisorModal
           open={showPendingAdvisorListModal}
           saboNumbers={allPendingAdvisorList!.map((reimbursementRequest) => reimbursementRequest.saboId!)}