style: added "Source Mailbox" and "Destination Mailbox" labels

Author: umanamente

nodes/Imap/operations/email/functions/EmailCopy.ts

@@ -2,84 +2,86 @@ import { CopyResponseObject, ImapFlow } from 'imapflow';
 import { IExecuteFunctions, INodeExecutionData, NodeApiError } from 'n8n-workflow';
 import { IResourceOperationDef } from '../../../utils/CommonDefinitions';
 import {
-	getMailboxPathFromNodeParameter,
-	parameterSelectMailbox,
+  getMailboxPathFromNodeParameter,
+  parameterSelectMailbox,
 } from '../../../utils/SearchFieldParameters';
 
 const PARAM_NAME_SOURCE_MAILBOX = 'sourceMailbox';
 const PARAM_NAME_DESTINATION_MAILBOX = 'destinationMailbox';
 
 export const copyEmailOperation: IResourceOperationDef = {
-	operation: {
-		name: 'Copy',
-		value: 'copyEmail',
-	},
-	parameters: [
-		{
-			...parameterSelectMailbox,
-			description: 'Select the mailbox',
-			name: PARAM_NAME_SOURCE_MAILBOX,
-		},
-		{
-			displayName: 'Email UID',
-			name: 'emailUid',
-			type: 'string',
-			default: '',
-			description: 'UID of the email to copy',
-			hint: 'You can use comma separated list of UIDs to copy multiple emails at once',
-		},
-		{
-			...parameterSelectMailbox,
-			description: 'Select the destination mailbox',
-			name: PARAM_NAME_DESTINATION_MAILBOX,
-		},
-	],
-	async executeImapAction(
-		context: IExecuteFunctions,
-		itemIndex: number,
-		client: ImapFlow,
-	): Promise<INodeExecutionData[] | null> {
-		var returnData: INodeExecutionData[] = [];
+  operation: {
+    name: 'Copy',
+    value: 'copyEmail',
+  },
+  parameters: [
+    {
+      ...parameterSelectMailbox,
+      displayName: 'Source Mailbox',
+      description: 'Select the source mailbox',
+      name: PARAM_NAME_SOURCE_MAILBOX,
+    },
+    {
+      displayName: 'Email UID',
+      name: 'emailUid',
+      type: 'string',
+      default: '',
+      description: 'UID of the email to copy',
+      hint: 'You can use comma separated list of UIDs to copy multiple emails at once',
+    },
+    {
+      ...parameterSelectMailbox,
+      displayName: 'Destination Mailbox',
+      description: 'Select the destination mailbox',
+      name: PARAM_NAME_DESTINATION_MAILBOX,
+    },
+  ],
+  async executeImapAction(
+    context: IExecuteFunctions,
+    itemIndex: number,
+    client: ImapFlow,
+  ): Promise<INodeExecutionData[] | null> {
+    var returnData: INodeExecutionData[] = [];
 
-		const sourceMailboxPath = getMailboxPathFromNodeParameter(
-			context,
-			itemIndex,
-			PARAM_NAME_SOURCE_MAILBOX,
-		);
-		const destinationMailboxPath = getMailboxPathFromNodeParameter(
-			context,
-			itemIndex,
-			PARAM_NAME_DESTINATION_MAILBOX,
-		);
+    const sourceMailboxPath = getMailboxPathFromNodeParameter(
+      context,
+      itemIndex,
+      PARAM_NAME_SOURCE_MAILBOX,
+    );
+    const destinationMailboxPath = getMailboxPathFromNodeParameter(
+      context,
+      itemIndex,
+      PARAM_NAME_DESTINATION_MAILBOX,
+    );
 
-		const emailUid = context.getNodeParameter('emailUid', itemIndex) as string;
+    const emailUid = context.getNodeParameter('emailUid', itemIndex) as string;
 
-		context.logger?.info(
-			`Copying email "${emailUid}" from "${sourceMailboxPath}" to "${destinationMailboxPath}"`,
-		);
+    context.logger?.info(
+      `Copying email "${emailUid}" from "${sourceMailboxPath}" to "${destinationMailboxPath}"`,
+    );
 
-		await client.mailboxOpen(sourceMailboxPath, { readOnly: false });
+    await client.mailboxOpen(sourceMailboxPath, { readOnly: false });
 
-		const resp: CopyResponseObject = await client.messageCopy(emailUid, destinationMailboxPath, {
-			uid: true,
-		});
+    const resp: CopyResponseObject = await client.messageCopy(emailUid, destinationMailboxPath, {
+      uid: true,
+    });
 
-		if (!resp || !resp.uidMap) {
-			throw new NodeApiError(
-				context.getNode(),
-				{},
-				{
-					message: 'Unable to copy email, unknown error',
-				},
-			);
-		}
+    if (!resp || !resp.uidMap) {
+      throw new NodeApiError(
+        context.getNode(),
+        {},
+        {
+          message: 'Unable to copy email, unknown error',
+        },
+      );
+    }
 
-		var item_json = JSON.parse(JSON.stringify(resp));
+    var item_json = JSON.parse(JSON.stringify(resp));
 
-		returnData.push({
-			json: item_json,
-		});
+    returnData.push({
+      json: item_json,
+    });
 
-		return returnData;
-	},
+    return returnData;
+  },
 };

nodes/Imap/operations/email/functions/EmailMove.ts

@@ -16,7 +16,8 @@ export const moveEmailOperation: IResourceOperationDef = {
   parameters: [
     {
       ...parameterSelectMailbox,
-      description: 'Select the mailbox',
+      displayName: 'Source Mailbox',
+      description: 'Select the source mailbox',
       name: PARAM_NAME_SOURCE_MAILBOX,
     },
     {
@@ -29,6 +30,7 @@ export const moveEmailOperation: IResourceOperationDef = {
     },
     {
       ...parameterSelectMailbox,
+      displayName: 'Destination Mailbox',
       description: 'Select the destination mailbox',
       name: PARAM_NAME_DESTINATION_MAILBOX,
     },