-
Couldn't load subscription status.
- Fork 402
feat(backend): Introduce OrganizationRoleAPI #2177
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
fe4f6aa
46aa684
7ca0d9a
fcc4cef
ae9e714
16d9e5d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| '@clerk/backend': patch | ||
| --- | ||
|
|
||
| Add OrganizationRoleAPI for CRUD operations regarding instance level organization roles. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,138 @@ | ||
| import { joinPaths } from '../../util/path'; | ||
| import type { DeletedObject, Role } from '../resources'; | ||
| import { AbstractAPI } from './AbstractApi'; | ||
|
|
||
| const basePath = '/organizations_roles'; | ||
|
|
||
| type GetRoleListParams = { | ||
| limit?: number; | ||
| offset?: number; | ||
| query?: string; | ||
| order_by?: string; | ||
| }; | ||
|
|
||
| type CreateParams = { | ||
| /** | ||
| * A name of a role in a readable friendly format. | ||
| * F.e. `Teacher` or `Administrator` | ||
| */ | ||
| name: string; | ||
|
|
||
| /** | ||
| * A unique identifier that represents the role. | ||
| * F.e. `org:administrator` | ||
| */ | ||
| key: string; | ||
|
|
||
| /** | ||
| * A brief description of what the role represents or its intended use. | ||
| */ | ||
| description: string; | ||
|
|
||
| /** | ||
| * An array of permission ids that will be assigned to this role. | ||
| */ | ||
| permissions: string[]; | ||
| }; | ||
|
|
||
| type GetOrganizationRoleParams = { roleId: string }; | ||
|
|
||
| type UpdateParams = { | ||
| /** | ||
| * A name of a role in a readable friendly format. | ||
| * F.e. `Teacher` or `Administrator` | ||
| * Passing undefined has no effect to the existing value. | ||
| */ | ||
| name?: string; | ||
|
|
||
| /** | ||
| * A unique identifier that represents the role. | ||
| * F.e. `org:administrator` | ||
| * Passing undefined has no effect to the existing value. | ||
| */ | ||
| key?: string; | ||
|
|
||
| /** | ||
| * A brief description of what the role represents or its intended use. | ||
| * Passing undefined has no effect to the existing value. | ||
| */ | ||
| description?: string; | ||
|
|
||
| /** | ||
| * An array of permission ids that will be assigned to this role. | ||
| * Passing undefined has no effect to the permission that already exist. | ||
| * Passing an empty array will override the existing permissions. | ||
| */ | ||
| permissions?: string[]; | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @chanioxaris when updating permissions, is it an addition or override ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It's an override. For additions, you have to use the dedicated endpoint. Of course, if you don't specify any property, we won't update them There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. { permissions: undefined } // will not override
{ permissions: [] } // will assign no perms to a role@chanioxaris is the above correct ? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's correct! |
||
| }; | ||
|
|
||
| type RemovePermissionParams = { | ||
| permissionId: string; | ||
| roleId: string; | ||
| }; | ||
|
|
||
| type AssignPermissionParams = RemovePermissionParams; | ||
|
|
||
| export class OrganizationRoleAPI extends AbstractAPI { | ||
| public async getOrganizationRoleList(params?: GetRoleListParams) { | ||
| return this.request<Role[]>({ | ||
| method: 'GET', | ||
| path: basePath, | ||
| queryParams: params, | ||
| }); | ||
| } | ||
|
|
||
| public async createOrganizationRole(params: CreateParams) { | ||
| return this.request<Role>({ | ||
| method: 'POST', | ||
| path: basePath, | ||
| bodyParams: params, | ||
| }); | ||
| } | ||
|
|
||
| public async getOrganizationRole(params: GetOrganizationRoleParams) { | ||
| this.requireId(params.roleId); | ||
|
|
||
| return this.request<Role>({ | ||
| method: 'GET', | ||
| path: joinPaths(basePath, params.roleId), | ||
| }); | ||
| } | ||
|
|
||
| public async updateOrganizationRole(roleId: string, params: UpdateParams) { | ||
| this.requireId(roleId); | ||
| return this.request<Role>({ | ||
| method: 'PATCH', | ||
| path: joinPaths(basePath, roleId), | ||
| bodyParams: params, | ||
| }); | ||
| } | ||
|
|
||
| public async deleteOrganizationRole(roleId: string) { | ||
| this.requireId(roleId); | ||
| return this.request<DeletedObject>({ | ||
| method: 'DELETE', | ||
| path: joinPaths(basePath, roleId), | ||
| }); | ||
| } | ||
|
|
||
| public async assignPermissionToRole(params: AssignPermissionParams) { | ||
| const { roleId, permissionId } = params; | ||
| this.requireId(roleId); | ||
| this.requireId(permissionId); | ||
| return this.request<Role>({ | ||
| method: 'POST', | ||
| path: joinPaths(basePath, roleId, 'permission', permissionId), | ||
| }); | ||
| } | ||
|
|
||
| public async removePermissionFromRole(params: RemovePermissionParams) { | ||
| const { roleId, permissionId } = params; | ||
| this.requireId(roleId); | ||
| this.requireId(permissionId); | ||
| return this.request<Role>({ | ||
| method: 'DELETE', | ||
| path: joinPaths(basePath, roleId, 'permission', permissionId), | ||
| }); | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,26 @@ | ||
| import type { RoleJSON } from './JSON'; | ||
| import { Permission } from './Permission'; | ||
|
|
||
| export class Role { | ||
| constructor( | ||
| readonly id: string, | ||
| readonly name: string, | ||
| readonly key: string, | ||
| readonly description: string, | ||
| readonly permissions: Permission[] = [], | ||
| readonly createdAt: number, | ||
| readonly updatedAt: number, | ||
| ) {} | ||
|
|
||
| static fromJSON(data: RoleJSON): Role { | ||
| return new Role( | ||
| data.id, | ||
| data.name, | ||
| data.key, | ||
| data.description, | ||
| (data.permissions || []).map(x => Permission.fromJSON(x)), | ||
| data.created_at, | ||
| data.updated_at, | ||
| ); | ||
| } | ||
| } |
Uh oh!
There was an error while loading. Please reload this page.