⚡ Improvements to Zoom-Node

packages/nodes-base/nodes/Zoom/GenericFunctions.ts

@@ -33,6 +33,7 @@ export async function zoomApiRequest(this: IExecuteFunctions | IExecuteSingleFun
 	if (Object.keys(query).length === 0) {
 		delete options.qs;
 	}
+
 	try {
 		if (authenticationMethod === 'accessToken') {
 			const credentials = this.getCredentials('zoomApi');

packages/nodes-base/nodes/Zoom/MeetingDescription.ts

@@ -43,31 +43,13 @@ export const meetingOperations = [
 		],
 		default: 'create',
 		description: 'The operation to perform.',
-	}
+	},
 ] as INodeProperties[];
 
 export const meetingFields = [
 	/* -------------------------------------------------------------------------- */
-	/*                                 meeting:create                                */
+	/*                                 meeting:create                             */
 	/* -------------------------------------------------------------------------- */
-	{
-		displayName: 'User ID',
-		name: 'userId',
-		type: 'string',
-		default: '',
-		required: true,
-		displayOptions: {
-			show: {
-				operation: [
-					'create',
-				],
-				resource: [
-					'meeting',
-				],
-			},
-		},
-		description: 'User ID or Email Address.',
-	},
 	{
 		displayName: 'Additional Fields',
 		name: 'additionalFields',
@@ -90,98 +72,29 @@ export const meetingFields = [
 				displayName: 'Agenda',
 				name: 'agenda',
 				type: 'string',
+				typeOptions: {
+					alwaysOpenEditWindow: true,
+				},
 				default: '',
 				description: 'Meeting agenda.',
 			},
-			{
-				displayName: 'Alternative Hosts',
-				name: 'alternativeHosts',
-				type: 'string',
-				default: '',
-				description: 'Alternative hosts email IDs.',
-			},
-			{
-				displayName: 'Auto Recording',
-				name: 'autoRecording',
-				type: 'options',
-				options: [
-					{
-						name: 'Record on Local',
-						value: 'local',
-					},
-					{
-						name: 'Record on Cloud',
-						value: 'cloud',
-					},
-					{
-						name: 'Disabled',
-						value: 'none',
-					},
-				],
-				default: 'none',
-				description: 'Auto recording.',
-			},
-			{
-				displayName: 'Audio',
-				name: 'audio',
-				type: 'options',
-				options: [
-					{
-						name: 'Both Telephony and VoiP',
-						value: 'both',
-					},
-					{
-						name: 'Telephony',
-						value: 'telephony',
-					},
-					{
-						name: 'VOIP',
-						value: 'voip',
-					},
-
-				],
-				default: 'both',
-				description: 'Determine how participants can join audio portion of the meeting.',
-			},
 			{
 				displayName: 'Duration',
 				name: 'duration',
 				type: 'number',
-				default: '',
+				typeOptions: {
-				description: 'Duration.',
+					minValue: 0,
-			},
+				},
-			{
+				default: 0,
-				displayName: 'Host Meeting in China',
+				description: 'Meeting duration (minutes).',
-				name: 'cnMeeting',
-				type: 'boolean',
-				default: false,
-				description: 'Host Meeting in China.',
-			},
-			{
-				displayName: 'Host Meeting in India',
-				name: 'inMeeting',
-				type: 'boolean',
-				default: false,
-				description: 'Host Meeting in India.',
-			},
-			{
-				displayName: 'Host Video',
-				name: 'hostVideo',
-				type: 'boolean',
-				default: false,
-				description: 'Start video when host joins the meeting.',
-			},
-			{
-				displayName: 'Join Before Host',
-				name: 'joinBeforeHost',
-				type: 'boolean',
-				default: false,
-				description: 'Allow participants to join the meeting before host starts it.',
 			},
 			{
 				displayName: 'Meeting Topic',
 				name: 'topic',
 				type: 'string',
+				typeOptions: {
+					alwaysOpenEditWindow: true,
+				},
 				default: '',
 				description: `Meeting topic.`,
 			},
@@ -211,20 +124,6 @@ export const meetingFields = [
 				default: 2,
 				description: 'Meeting type.',
 			},
-			{
-				displayName: 'Muting before entry',
-				name: 'muteUponEntry',
-				type: 'boolean',
-				default: false,
-				description: 'Mute participants upon entry.',
-			},
-			{
-				displayName: 'Participant Video',
-				name: 'participantVideo',
-				type: 'boolean',
-				default: false,
-				description: 'Start video when participant joins the meeting.',
-			},
 			{
 				displayName: 'Password',
 				name: 'password',
@@ -232,27 +131,6 @@ export const meetingFields = [
 				default: '',
 				description: 'Password to join the meeting with maximum 10 characters.',
 			},
-			{
-				displayName: 'Registration Type',
-				name: 'registrationType',
-				type: 'options',
-				options: [
-					{
-						name: 'Attendees register once and can attend any of the occurences',
-						value: 1,
-					},
-					{
-						name: 'Attendees need to register for every occurrence',
-						value: 2,
-					},
-					{
-						name: 'Attendees register once and can choose one or more occurrences to attend',
-						value: 3,
-					},
-				],
-				default: 1,
-				description: 'Registration type. Used for recurring meetings with fixed time only',
-			},
 			{
 				displayName: 'Schedule For',
 				name: 'scheduleFor',
@@ -260,6 +138,135 @@ export const meetingFields = [
 				default: '',
 				description: 'Schedule meeting for someone else from your account, provide their email ID.',
 			},
+			{
+				displayName: 'Settings',
+				name: 'settings',
+				type: 'collection',
+				placeholder: 'Add Setting',
+				default: {},
+				options: [
+					{
+						displayName: 'Audio',
+						name: 'audio',
+						type: 'options',
+						options: [
+							{
+								name: 'Both Telephony and VoiP',
+								value: 'both',
+							},
+							{
+								name: 'Telephony',
+								value: 'telephony',
+							},
+							{
+								name: 'VOIP',
+								value: 'voip',
+							},
+
+						],
+						default: 'both',
+						description: 'Determine how participants can join audio portion of the meeting.',
+					},
+					{
+						displayName: 'Alternative Hosts',
+						name: 'alternativeHosts',
+						type: 'string',
+						default: '',
+						description: 'Alternative hosts email IDs.',
+					},
+					{
+						displayName: 'Auto Recording',
+						name: 'autoRecording',
+						type: 'options',
+						options: [
+							{
+								name: 'Record on Local',
+								value: 'local',
+							},
+							{
+								name: 'Record on Cloud',
+								value: 'cloud',
+							},
+							{
+								name: 'Disabled',
+								value: 'none',
+							},
+						],
+						default: 'none',
+						description: 'Auto recording.',
+					},
+					{
+						displayName: 'Host Meeting in China',
+						name: 'cnMeeting',
+						type: 'boolean',
+						default: false,
+						description: 'Host Meeting in China.',
+					},
+					{
+						displayName: 'Host Meeting in India',
+						name: 'inMeeting',
+						type: 'boolean',
+						default: false,
+						description: 'Host Meeting in India.',
+					},
+					{
+						displayName: 'Host Video',
+						name: 'hostVideo',
+						type: 'boolean',
+						default: false,
+						description: 'Start video when host joins the meeting.',
+					},
+					{
+						displayName: 'Join Before Host',
+						name: 'joinBeforeHost',
+						type: 'boolean',
+						default: false,
+						description: 'Allow participants to join the meeting before host starts it.',
+					},
+					{
+						displayName: 'Muting Upon Entry',
+						name: 'muteUponEntry',
+						type: 'boolean',
+						default: false,
+						description: 'Mute participants upon entry.',
+					},
+					{
+						displayName: 'Participant Video',
+						name: 'participantVideo',
+						type: 'boolean',
+						default: false,
+						description: 'Start video when participant joins the meeting.',
+					},
+					{
+						displayName: 'Registration Type',
+						name: 'registrationType',
+						type: 'options',
+						options: [
+							{
+								name: 'Attendees register once and can attend any of the occurences',
+								value: 1,
+							},
+							{
+								name: 'Attendees need to register for every occurrence',
+								value: 2,
+							},
+							{
+								name: 'Attendees register once and can choose one or more occurrences to attend',
+								value: 3,
+							},
+						],
+						default: 1,
+						description: 'Registration type. Used for recurring meetings with fixed time only',
+					},
+					{
+						displayName: 'Watermark',
+						name: 'watermark',
+						type: 'boolean',
+						default: false,
+						description: 'Adds watermark when viewing a shared screen.',
+					},
+				],
+			},
 			{
 				displayName: 'Start Time',
 				name: 'startTime',
@@ -277,13 +284,6 @@ export const meetingFields = [
 				default: '',
 				description: `Time zone used in the response. The default is the time zone of the calendar.`,
 			},
-			{
-				displayName: 'Watermark',
-				name: 'watermark',
-				type: 'boolean',
-				default: false,
-				description: 'Adds watermark when viewing a shared screen.',
-			},
 		],
 	},
 	/* -------------------------------------------------------------------------- */
@@ -342,26 +342,8 @@ export const meetingFields = [
 		],
 	},
 	/* -------------------------------------------------------------------------- */
-	/*                                 meeting:getAll                               */
+	/*                                 meeting:getAll                             */
 	/* -------------------------------------------------------------------------- */
-	{
-		displayName: 'User ID',
-		name: 'userId',
-		type: 'string',
-		default: '',
-		required: true,
-		displayOptions: {
-			show: {
-				operation: [
-					'getAll',
-				],
-				resource: [
-					'meeting',
-				],
-			},
-		},
-		description: 'User ID or email-ID.',
-	},
 	{
 		displayName: 'Return All',
 		name: 'returnAll',
@@ -404,10 +386,10 @@ export const meetingFields = [
 		description: 'How many results to return.',
 	},
 	{
-		displayName: 'Additional Fields',
+		displayName: 'Filters',
-		name: 'additionalFields',
+		name: 'filters',
 		type: 'collection',
-		placeholder: 'Add Field',
+		placeholder: 'Add Filter',
 		default: {},
 		displayOptions: {
 			show: {
@@ -429,23 +411,26 @@ export const meetingFields = [
 					{
 						name: 'Scheduled',
 						value: 'scheduled',
+						description: 'This includes all valid past meetings, live meetings and upcoming scheduled meetings'
 					},
 					{
 						name: 'Live',
 						value: 'live',
+						description: 'All ongoing meetings',
 					},
 					{
 						name: 'Upcoming',
 						value: 'upcoming',
+						description: 'All upcoming meetings including live meetings',
 					},
 				],
 				default: 'live',
 				description: `Meeting type.`,
 			},
-		]
+		],
 	},
 	/* -------------------------------------------------------------------------- */
-	/*                                 meeting:delete                                */
+	/*                                 meeting:delete                             */
 	/* -------------------------------------------------------------------------- */
 	{
 		displayName: 'Meeting ID',
@@ -500,7 +485,7 @@ export const meetingFields = [
 
 	},
 	/* -------------------------------------------------------------------------- */
-	/*                                 meeting:update                                */
+	/*                                 meeting:update                             */
 	/* -------------------------------------------------------------------------- */
 	{
 		displayName: 'Meeting ID',
@@ -521,8 +506,8 @@ export const meetingFields = [
 		description: 'Meeting ID.',
 	},
 	{
-		displayName: 'Additional Fields',
+		displayName: 'Update Fields',
-		name: 'additionalFields',
+		name: 'updateFields',
 		type: 'collection',
 		placeholder: 'Add Field',
 		default: {},
@@ -541,100 +526,21 @@ export const meetingFields = [
 				displayName: 'Agenda',
 				name: 'agenda',
 				type: 'string',
+				typeOptions: {
+					alwaysOpenEditWindow: true,
+				},
 				default: '',
 				description: 'Meeting agenda.',
 			},
-			{
-				displayName: 'Alternative Hosts',
-				name: 'alternativeHosts',
-				type: 'string',
-				default: '',
-				description: 'Alternative hosts email IDs.',
-			},
-			{
-				displayName: 'Audio',
-				name: 'audio',
-				type: 'options',
-				options: [
-					{
-						name: 'Both Telephony and VoiP',
-						value: 'both',
-					},
-					{
-						name: 'Telephony',
-						value: 'telephony',
-					},
-					{
-						name: 'VOIP',
-						value: 'voip',
-					},
-				],
-				default: 'both',
-				description: 'Determine how participants can join audio portion of the meeting.',
-			},
-			{
-				displayName: 'Auto Recording',
-				name: 'autoRecording',
-				type: 'options',
-				options: [
-					{
-						name: 'Record on Local',
-						value: 'local',
-					},
-					{
-						name: 'Record on Cloud',
-						value: 'cloud',
-					},
-					{
-						name: 'Disabled',
-						value: 'none',
-					},
-				],
-				default: 'none',
-				description: 'Auto recording.',
-			},
-
 			{
 				displayName: 'Duration',
 				name: 'duration',
 				type: 'number',
-				default: '',
+				typeOptions: {
-				description: 'Duration.',
+					minValue: 0,
-			},
+				},
-			{
+				default: 0,
-				displayName: 'Join Before Host',
+				description: 'Meeting duration (minutes).',
-				name: 'joinBeforeHost',
-				type: 'boolean',
-				default: false,
-				description: 'Allow participants to join the meeting before host starts it.',
-			},
-			{
-				displayName: 'Host Meeting in China',
-				name: 'cnMeeting',
-				type: 'boolean',
-				default: false,
-				description: 'Host Meeting in China.',
-			},
-			{
-				displayName: 'Host Meeting in India',
-				name: 'inMeeting',
-				type: 'boolean',
-				default: false,
-				description: 'Host Meeting in India.',
-			},
-			{
-				displayName: 'Host Video',
-				name: 'hostVideo',
-				type: 'boolean',
-				default: false,
-				description: 'Start video when host joins the meeting.',
-			},
-			{
-				displayName: 'Occurrence ID',
-				name: 'occurrenceId',
-				type: 'string',
-				default: '',
-				description: 'Occurrence ID.',
 			},
 			{
 				displayName: 'Meeting Topic',
@@ -669,13 +575,6 @@ export const meetingFields = [
 				default: 2,
 				description: 'Meeting type.',
 			},
-			{
-				displayName: 'Muting Before Entry',
-				name: 'muteUponEntry',
-				type: 'boolean',
-				default: false,
-				description: 'Mute participants upon entry.',
-			},
 			{
 				displayName: 'Password',
 				name: 'password',
@@ -683,34 +582,6 @@ export const meetingFields = [
 				default: '',
 				description: 'Password to join the meeting with maximum 10 characters.',
 			},
-			{
-				displayName: 'Participant Video',
-				name: 'participantVideo',
-				type: 'boolean',
-				default: false,
-				description: 'Start video when participant joins the meeting.',
-			},
-			{
-				displayName: 'Registration Type',
-				name: 'registrationType',
-				type: 'options',
-				options: [
-					{
-						name: 'Attendees register once and can attend any of the occurrences',
-						value: 1,
-					},
-					{
-						name: 'Attendees need to register for every occurrence',
-						value: 2,
-					},
-					{
-						name: 'Attendees register once and can choose one or more occurrences to attend',
-						value: 3,
-					},
-				],
-				default: 1,
-				description: 'Registration type. Used for recurring meetings with fixed time only',
-			},
 			{
 				displayName: 'Schedule For',
 				name: 'scheduleFor',
@@ -718,6 +589,135 @@ export const meetingFields = [
 				default: '',
 				description: 'Schedule meeting for someone else from your account, provide their email ID.',
 			},
+			{
+				displayName: 'Settings',
+				name: 'settings',
+				type: 'collection',
+				placeholder: 'Add Setting',
+				default: {},
+				options: [
+					{
+						displayName: 'Audio',
+						name: 'audio',
+						type: 'options',
+						options: [
+							{
+								name: 'Both Telephony and VoiP',
+								value: 'both',
+							},
+							{
+								name: 'Telephony',
+								value: 'telephony',
+							},
+							{
+								name: 'VOIP',
+								value: 'voip',
+							},
+
+						],
+						default: 'both',
+						description: 'Determine how participants can join audio portion of the meeting.',
+					},
+					{
+						displayName: 'Alternative Hosts',
+						name: 'alternativeHosts',
+						type: 'string',
+						default: '',
+						description: 'Alternative hosts email IDs.',
+					},
+					{
+						displayName: 'Auto Recording',
+						name: 'autoRecording',
+						type: 'options',
+						options: [
+							{
+								name: 'Record on Local',
+								value: 'local',
+							},
+							{
+								name: 'Record on Cloud',
+								value: 'cloud',
+							},
+							{
+								name: 'Disabled',
+								value: 'none',
+							},
+						],
+						default: 'none',
+						description: 'Auto recording.',
+					},
+					{
+						displayName: 'Host Meeting in China',
+						name: 'cnMeeting',
+						type: 'boolean',
+						default: false,
+						description: 'Host Meeting in China.',
+					},
+					{
+						displayName: 'Host Meeting in India',
+						name: 'inMeeting',
+						type: 'boolean',
+						default: false,
+						description: 'Host Meeting in India.',
+					},
+					{
+						displayName: 'Host Video',
+						name: 'hostVideo',
+						type: 'boolean',
+						default: false,
+						description: 'Start video when host joins the meeting.',
+					},
+					{
+						displayName: 'Join Before Host',
+						name: 'joinBeforeHost',
+						type: 'boolean',
+						default: false,
+						description: 'Allow participants to join the meeting before host starts it.',
+					},
+					{
+						displayName: 'Muting Upon Entry',
+						name: 'muteUponEntry',
+						type: 'boolean',
+						default: false,
+						description: 'Mute participants upon entry.',
+					},
+					{
+						displayName: 'Participant Video',
+						name: 'participantVideo',
+						type: 'boolean',
+						default: false,
+						description: 'Start video when participant joins the meeting.',
+					},
+					{
+						displayName: 'Registration Type',
+						name: 'registrationType',
+						type: 'options',
+						options: [
+							{
+								name: 'Attendees register once and can attend any of the occurences',
+								value: 1,
+							},
+							{
+								name: 'Attendees need to register for every occurrence',
+								value: 2,
+							},
+							{
+								name: 'Attendees register once and can choose one or more occurrences to attend',
+								value: 3,
+							},
+						],
+						default: 1,
+						description: 'Registration type. Used for recurring meetings with fixed time only',
+					},
+					{
+						displayName: 'Watermark',
+						name: 'watermark',
+						type: 'boolean',
+						default: false,
+						description: 'Adds watermark when viewing a shared screen.',
+					},
+				],
+			},
 			{
 				displayName: 'Start Time',
 				name: 'startTime',
@@ -735,16 +735,6 @@ export const meetingFields = [
 				default: '',
 				description: `Time zone used in the response. The default is the time zone of the calendar.`,
 			},
-			{
-				displayName: 'Watermark',
-				name: 'watermark',
-				type: 'boolean',
-				default: false,
-				description: 'Adds watermark when viewing a shared screen.',
-			},
-
-
 		],
 	},
-
 ] as INodeProperties[];

packages/nodes-base/nodes/Zoom/MeetingRegistrantDescription.ts

@@ -1,6 +1,7 @@
 import {
 	INodeProperties,
 } from 'n8n-workflow';
+
 export const meetingRegistrantOperations = [
 	{
 		displayName: 'Operation',
@@ -9,7 +10,7 @@ export const meetingRegistrantOperations = [
 		displayOptions: {
 			show: {
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -38,7 +39,7 @@ export const meetingRegistrantOperations = [
 
 export const meetingRegistrantFields = [
 	/* -------------------------------------------------------------------------- */
-	/*                                 meetingRegistrants:create                                */
+	/*                                 meetingRegistrant:create                   */
 	/* -------------------------------------------------------------------------- */
 	{
 		displayName: 'Meeting Id',
@@ -52,7 +53,7 @@ export const meetingRegistrantFields = [
 					'create',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -70,7 +71,7 @@ export const meetingRegistrantFields = [
 					'create',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -88,7 +89,7 @@ export const meetingRegistrantFields = [
 					'create',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -107,7 +108,7 @@ export const meetingRegistrantFields = [
 
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -248,7 +249,7 @@ export const meetingRegistrantFields = [
 		],
 	},
 	/* -------------------------------------------------------------------------- */
-	/*                                 meetingRegistrants:getAll                                */
+	/*                                 meetingRegistrant:getAll                   */
 	/* -------------------------------------------------------------------------- */
 	{
 		displayName: 'Meeting ID',
@@ -262,7 +263,7 @@ export const meetingRegistrantFields = [
 					'getAll',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -278,7 +279,7 @@ export const meetingRegistrantFields = [
 					'getAll',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -295,7 +296,7 @@ export const meetingRegistrantFields = [
 					'getAll',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 				returnAll: [
 					false,
@@ -322,7 +323,7 @@ export const meetingRegistrantFields = [
 
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -359,7 +360,7 @@ export const meetingRegistrantFields = [
 		]
 	},
 	/* -------------------------------------------------------------------------- */
-	/*                                 meetingRegistrants:update                                */
+	/*                                 meetingRegistrant:update                   */
 	/* -------------------------------------------------------------------------- */
 	{
 		displayName: 'Meeting ID',
@@ -373,7 +374,7 @@ export const meetingRegistrantFields = [
 					'update',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -390,7 +391,7 @@ export const meetingRegistrantFields = [
 					'update',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},
@@ -423,7 +424,7 @@ export const meetingRegistrantFields = [
 					'update',
 				],
 				resource: [
-					'meetingRegistrants',
+					'meetingRegistrant',
 				],
 			},
 		},