Skip to content

Commit 4c7bb93

Browse files
#1425: use isFinanceLead in frontend
1 parent ec58f3e commit 4c7bb93

2 files changed

Lines changed: 2 additions & 4 deletions

File tree

src/frontend/src/pages/AdminToolsPage/AdminToolsPage.tsx

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,7 @@ const AdminToolsPage: React.FC = () => {
1717
<PageLayout title="Admin Tools">
1818
{isHead(currentUser.role) && <AdminToolsUserManagement />}
1919
{isAdmin(currentUser.role) && <AdminToolsSlackUpcomingDeadlines />}
20-
{(isAdmin(currentUser.role) || (isLeadership(currentUser.role) && currentUser.isFinance)) && (
21-
<AdminToolsFinanceConfig />
22-
)}
20+
{(isAdmin(currentUser.role) || currentUser.isFinanceLead) && <AdminToolsFinanceConfig />}
2321
</PageLayout>
2422
);
2523
};

src/frontend/src/utils/users.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,5 +17,5 @@ export const userToAutocompleteOption = (user: User): { label: string; id: numbe
1717
* @returns whether they can view Admin Tools
1818
*/
1919
export const canAccessAdminTools = (user?: AuthenticatedUser): boolean => {
20-
return isHead(user?.role) || (isLeadership(user?.role) && user?.isFinance) || false;
20+
return isHead(user?.role) || user?.isFinanceLead || false;
2121
};

0 commit comments

Comments
 (0)