Skip to content

Commit c4f07a4

Browse files
committed
#2128 clean up some comments
1 parent 0f9d0fd commit c4f07a4

1 file changed

Lines changed: 2 additions & 26 deletions

File tree

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

Lines changed: 2 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -18,28 +18,8 @@ import { routes } from '../../utils/routes';
1818

1919
const AdminToolsPage: React.FC = () => {
2020
const currentUser = useCurrentUser();
21-
2221
const [tabIndex, setTabIndex] = useState<number>(0);
2322

24-
/*
25-
User Management Tab
26-
{isHead(currentUser.role) && <AdminToolsUserManagement />}
27-
{isAdmin(currentUser.role) && <TeamsTools />}
28-
29-
Project Configuration Tab
30-
{isAdmin(currentUser.role) && <AdminToolsBOMConfig />}
31-
{isHead(currentUser.role) && <AdminToolsProjectsConfig />}
32-
33-
34-
35-
Finance Configuration Tab
36-
{(isAdmin(currentUser.role) || currentUser.isAtLeastFinanceLead) && <AdminToolsFinanceConfig />}
37-
38-
39-
Miscellaneous Tab
40-
{isAdmin(currentUser.role) && <AdminToolsSlackUpcomingDeadlines />}
41-
*/
42-
4323
const isUserHead = isHead(currentUser.role);
4424
const isUserAdmin = isAdmin(currentUser.role);
4525
const isUserFinanceLead = currentUser.isAtLeastFinanceLead;
@@ -58,17 +38,13 @@ const AdminToolsPage: React.FC = () => {
5838
if (isUserAdmin) {
5939
tabs.push({ tabUrlValue: 'miscellaneous', tabName: 'Miscellaneous' });
6040
}
61-
// { tabUrlValue: 'user-management', tabName: 'User Management', roles: ['head', 'admin'] },
62-
// { tabUrlValue: 'project-configuration', tabName: 'Project Configuration', roles: ['head', 'admin'] },
63-
// { tabUrlValue: 'finance-configuration', tabName: 'Finance Configuration', roles: ['financeLead', 'admin'] },
64-
// { tabUrlValue: 'miscellaneous', tabName: 'Miscellaneous', roles: ['admin'] }
6541

6642
const showUserManagement = () => {
67-
return isUserAdmin ? <TeamsTools /> : <AdminToolsUserManagement />;
43+
return isUserAdmin ? <TeamsTools /> : isUserHead && <AdminToolsUserManagement />;
6844
};
6945

7046
const showProjectConfiguration = () => {
71-
return isUserAdmin ? <AdminToolsBOMConfig /> : <AdminToolsProjectsConfig />;
47+
return isUserAdmin ? <AdminToolsBOMConfig /> : isUserHead && <AdminToolsProjectsConfig />;
7248
};
7349

7450
return (

0 commit comments

Comments
 (0)