fix: display all roles if we can't get project roles

Unleash · Feb 20, 2024 · 7350c91 · 7350c91
1 parent b22b423
commit 7350c91
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/frontend/src/component/application/ApplicationEdit/ApplicationEdit.tsx b/frontend/src/component/application/ApplicationEdit/ApplicationEdit.tsx
@@ -16,7 +16,6 @@ import { ConditionallyRender } from 'component/common/ConditionallyRender/Condit
 import { UPDATE_APPLICATION } from 'component/providers/AccessProvider/permissions';
 import { ApplicationView } from '../ApplicationView/ApplicationView';
 import { ApplicationUpdate } from '../ApplicationUpdate/ApplicationUpdate';
-import { ConnectedInstances } from '../ConnectedInstances/ConnectedInstances';
 import { Dialogue } from 'component/common/Dialogue/Dialogue';
 import { PageContent } from 'component/common/PageContent/PageContent';
 import { PageHeader } from 'component/common/PageHeader/PageHeader';

diff --git a/frontend/src/component/project/ProjectAccess/ProjectAccessAssign/ProjectAccessAssign.tsx b/frontend/src/component/project/ProjectAccess/ProjectAccessAssign/ProjectAccessAssign.tsx
@@ -321,13 +321,20 @@ export const ProjectAccessAssign = ({
     };
 
     const isValid = selectedOptions.length > 0 && selectedRoles.length > 0;
-    const filteredRoles = userRoles.some(
-        (userrole) => userrole.name === 'Admin' || userrole.name === 'Owner',
-    )
-        ? roles
-        : roles.filter((role) =>
-              userRoles.some((userrole) => role.id === userrole.id),
-          );
+    const displayAllRoles =
+        userRoles.length === 0 ||
+        userRoles.some(
+            (userRole) =>
+                userRole.name === 'Admin' || userRole.name === 'Owner',
+        );
+    let filteredRoles: IRole[];
+    if (displayAllRoles) {
+        filteredRoles = roles;
+    } else {
+        filteredRoles = roles.filter((role) =>
+            userRoles.some((userrole) => role.id === userrole.id),
+        );
+    }
     return (
         <SidebarModal
             open

diff --git a/src/lib/routes/admin-api/user/user.ts b/src/lib/routes/admin-api/user/user.ts
@@ -185,6 +185,7 @@ class UserController extends Controller {
                 },
             );
         } else {
+            res.status(400).end();
         }
     }
     async getMe(req: IAuthRequest, res: Response<MeSchema>): Promise<void> {