Skip to content

Commit efff0db

Browse files
authored
Fix 403 from organizations endpoint when retrieving users (#1437)
1 parent 19fff6f commit efff0db

File tree

3 files changed

+10
-12
lines changed

3 files changed

+10
-12
lines changed

web_ui/src/pages/user-management/user-management.component.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ export const UserManagement = () => {
151151
id: `${UserManagementTabs.USERS}-user-page`,
152152
key: UserManagementTabs.USERS,
153153
name: capitalize(UserManagementTabs.USERS),
154-
children: <UsersTab activeUser={activeUser as User} />,
154+
children: <UsersTab activeUser={activeUser as User} resourceId={organizationId} />,
155155
},
156156
{
157157
id: `${UserManagementTabs.PERSONAL_ACCESS_TOKEN}-user-page`,

web_ui/src/pages/user-management/users/users-tab.component.tsx

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,15 @@
44
import { RESOURCE_TYPE, User } from '@geti/core/src/users/users.interface';
55
import { Flex, Text } from '@geti/ui';
66

7-
import { useOrganization } from '../../../intel-admin-app/pages/organization/hooks/organization.hook';
87
import { OrganizationUserActions } from './actions/organization-user-actions.component';
98
import { Users } from './users.component';
109

1110
interface UsersTabProps {
1211
activeUser: User | undefined;
12+
resourceId: string;
1313
}
1414

15-
export const UsersTab = ({ activeUser }: UsersTabProps) => {
16-
const { organizationId } = useOrganization();
17-
15+
export const UsersTab = ({ activeUser, resourceId }: UsersTabProps) => {
1816
if (!activeUser) return <></>;
1917

2018
return (
@@ -28,7 +26,7 @@ export const UsersTab = ({ activeUser }: UsersTabProps) => {
2826
<Users
2927
activeUser={activeUser}
3028
resourceType={RESOURCE_TYPE.ORGANIZATION}
31-
resourceId={organizationId}
29+
resourceId={resourceId}
3230
UserActions={OrganizationUserActions}
3331
/>
3432
</Flex>

web_ui/src/pages/user-management/users/users-tab.test.tsx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ describe('UsersTab', () => {
4545
platformUtilsService.getProductInfo = async () =>
4646
Promise.resolve(getMockedProductInfo({ isSmtpDefined: false }));
4747

48-
await render(<UsersTab activeUser={adminUser} />, {
48+
await render(<UsersTab activeUser={adminUser} resourceId='org-id' />, {
4949
services: { platformUtilsService, usersService },
5050
});
5151

@@ -58,7 +58,7 @@ describe('UsersTab', () => {
5858
platformUtilsService.getProductInfo = async () =>
5959
Promise.resolve(getMockedProductInfo({ isSmtpDefined: true }));
6060

61-
await render(<UsersTab activeUser={adminUser} />, {
61+
await render(<UsersTab activeUser={adminUser} resourceId='org-id' />, {
6262
services: { platformUtilsService, usersService },
6363
});
6464

@@ -76,7 +76,7 @@ describe('UsersTab', () => {
7676
platformUtilsService.getProductInfo = async () =>
7777
Promise.resolve(getMockedProductInfo({ isSmtpDefined: false }));
7878

79-
await render(<UsersTab activeUser={contributorUser} />, {
79+
await render(<UsersTab activeUser={contributorUser} resourceId='org-id' />, {
8080
services: { platformUtilsService, usersService },
8181
});
8282

@@ -89,7 +89,7 @@ describe('UsersTab', () => {
8989
platformUtilsService.getProductInfo = async () =>
9090
Promise.resolve(getMockedProductInfo({ isSmtpDefined: true }));
9191

92-
await render(<UsersTab activeUser={contributorUser} />, {
92+
await render(<UsersTab activeUser={contributorUser} resourceId='org-id' />, {
9393
services: { platformUtilsService, usersService },
9494
});
9595

@@ -113,7 +113,7 @@ describe('UsersTab', () => {
113113
platformUtilsService.getProductInfo = async () =>
114114
Promise.resolve(getMockedProductInfo({ isSmtpDefined: false }));
115115

116-
await render(<UsersTab activeUser={adminUser} />, {
116+
await render(<UsersTab activeUser={adminUser} resourceId='org-id' />, {
117117
services: { platformUtilsService, usersService },
118118
});
119119

@@ -131,7 +131,7 @@ describe('UsersTab', () => {
131131
platformUtilsService.getProductInfo = async () =>
132132
Promise.resolve(getMockedProductInfo({ isSmtpDefined: true }));
133133

134-
await render(<UsersTab activeUser={contributorUser} />, {
134+
await render(<UsersTab activeUser={contributorUser} resourceId='org-id' />, {
135135
services: { platformUtilsService, usersService },
136136
});
137137

0 commit comments

Comments
 (0)