server: remove deprecated permissions field from `GET /api/room/:ro…

…om` endpoint (#1768)
dyc3 · Apr 30, 2024 · a63aa8c · a63aa8c
1 parent cf03f77
commit a63aa8c
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 19 deletions.
diff --git a/client/src/components/RoomSettingsForm.vue b/client/src/components/RoomSettingsForm.vue
@@ -186,15 +186,7 @@ onMounted(async () => {
 
 function intoSettings(obj: OttApiResponseGetRoom): RoomSettings {
 	return {
-		..._.omit(obj, [
-			"name",
-			"isTemporary",
-			"users",
-			"queue",
-			"permissions",
-			"hasOwner",
-			"grants",
-		]),
+		..._.omit(obj, ["name", "isTemporary", "users", "queue", "hasOwner", "grants"]),
 		grants: new Grants(obj.grants),
 	};
 }

diff --git a/common/models/rest-api.ts b/common/models/rest-api.ts
@@ -62,8 +62,6 @@ export interface OttApiResponseGetRoom extends RoomSettings {
 	hasOwner: boolean;
 	grants: Grants;
 	autoSkipSegmentCategories: Category[];
-	/** @deprecated */
-	permissions: Grants;
 	restoreQueueBehavior: BehaviorOption;
 	users: RoomUserInfo[];
 }

diff --git a/docs/api.yaml b/docs/api.yaml
@@ -136,12 +136,6 @@ paths:
                       queue:
                         type: array
                         items: {}
-                      permissions:
-                        type: array
-                        items:
-                          type: array
-                          items:
-                            type: number
                       name:
                         type: string
                       isTemporary:
@@ -200,7 +194,7 @@ paths:
                       $ref: "#/components/schemas/visibility"
                     queueMode:
                       $ref: "#/components/schemas/queuemode"
-                    autoSkipSegmentCategories: 
+                    autoSkipSegmentCategories:
                       $ref: "#/components/schemas/autoSkipSegmentCategories"
                     grants:
                       description: Map of role numbers to the bitmask of permissions that role has

diff --git a/server/api/room.ts b/server/api/room.ts
@@ -182,7 +182,6 @@ const getRoom: RequestHandler<{ name: string }, OttApiResponseGetRoom, unknown>
 			])
 		),
 		queue: room.queue.items,
-		permissions: room.grants,
 		hasOwner: !!room.owner,
 	};
 	res.json(resp);