refactor: Add TODO comments for direct room restrictions and error handling in user invitation

Author: ggazzo

apps/meteor/app/lib/server/methods/addUsersToRoom.ts

@@ -52,7 +52,7 @@ export const addUsersToRoomMethod = async (userId: string, data: { rid: string;
 	});
 	const userInRoom = subscription != null;
 
-	// Can't add to direct room ever
+	// TODO: Can't add to direct room ever, unless it's a federated room
 	if (room.t === 'd') {
 		throw new Meteor.Error('error-cant-invite-for-direct-room', "Can't invite user to direct rooms", {
 			method: 'addUsersToRoom',

apps/meteor/app/slashcommands-invite/server/server.ts

@@ -96,6 +96,7 @@ slashCommands.add({
 		await Promise.all(
 			usersFiltered.map(async (user) => {
 				try {
+					// TODO: Refactor this to return an error if some user fails to be added
 					return await addUsersToRoomMethod(
 						userId,
 						{