remove redundant logic and add minLength 1 to the Subscriptions GetOne Schema

Author: ahmed-n-abdeltwab

apps/meteor/app/api/server/v1/subscriptions.ts

@@ -55,6 +55,7 @@ const SubscriptionsGetOneSchema = {
 	properties: {
 		roomId: {
 			type: 'string',
+			minLength: 1,
 		},
 	},
 	required: ['roomId'],
@@ -200,10 +201,6 @@ const subscriptionsEndpoints = API.v1.get(
 	async function action() {
 		const { roomId } = this.queryParams;
 
-		if (!roomId) {
-			return API.v1.failure("The 'roomId' param is required");
-		}
-
 		return API.v1.success({
 			subscription: await Subscriptions.findOneByRoomIdAndUserId(roomId, this.userId),
 		});