Skip to content

Commit 07fa234

Browse files
committed
fix: correct API endpoint paths and update authorization scope format
1 parent 7ec27d8 commit 07fa234

File tree

2 files changed

+5
-10
lines changed

2 files changed

+5
-10
lines changed

src/authz-module/data/api.ts

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
2-
import { LibraryMetadata, TeamMember, TeamRole } from '@src/types';
2+
import { LibraryMetadata, TeamMember } from '@src/types';
33
import { camelCaseObject } from '@edx/frontend-platform';
44
import { getApiUrl, getStudioApiUrl } from '@src/data/utils';
55

@@ -26,7 +26,7 @@ export interface AddTeamMembersRequest {
2626

2727
// TODO: replece api path once is created
2828
export const getTeamMembers = async (object: string): Promise<TeamMember[]> => {
29-
const { data } = await getAuthenticatedHttpClient().get(getApiUrl(`/api/authz/v1/roles/users?scope=${object}`));
29+
const { data } = await getAuthenticatedHttpClient().get(getApiUrl(`/api/authz/v1/roles/users/?scope=${object}`));
3030
return camelCaseObject(data.results);
3131
};
3232

@@ -37,11 +37,6 @@ export const addTeamMembers = async (
3737
return camelCaseObject(res.data);
3838
};
3939

40-
export const getTeamRoles = async (libraryId: string): Promise<TeamRole[]> => {
41-
const { data } = await getAuthenticatedHttpClient().get(getApiUrl(`/api/authz/v1/roles/?scope=${libraryId}`));
42-
return data;
43-
};
44-
4540
// TODO: this should be replaced in the future with Console API
4641
export const getLibrary = async (libraryId: string): Promise<LibraryMetadata> => {
4742
const { data } = await getAuthenticatedHttpClient().get(getStudioApiUrl(`/api/libraries/v2/${libraryId}/`));
@@ -54,8 +49,8 @@ export const getLibrary = async (libraryId: string): Promise<LibraryMetadata> =>
5449
};
5550

5651
export const getPermissionsByRole = async (scope: string): Promise<PermissionsByRole[]> => {
57-
const url = new URL(getApiUrl('/api/authz/v1/roles'));
52+
const url = new URL(getApiUrl('/api/authz/v1/roles/'));
5853
url.searchParams.append('scope', scope);
5954
const { data } = await getAuthenticatedHttpClient().get(url);
60-
return camelCaseObject(data);
55+
return camelCaseObject(data.results);
6156
};

src/authz-module/libraries-manager/context.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { PermissionMetadata, ResourceMetadata, Role } from 'types';
99
import { libraryPermissions, libraryResourceTypes, libraryRolesMetadata } from './constants';
1010

1111
const LIBRARY_TEAM_PERMISSIONS = ['view_library_team', 'manage_library_team'];
12-
const LIBRARY_AUTHZ_SCOPE = 'lib:*';
12+
const LIBRARY_AUTHZ_SCOPE = '*';
1313

1414
export type AppContextType = {
1515
authenticatedUser: {

0 commit comments

Comments
 (0)