remove empty fields when building message object

Author: nazabucciarelli

.changeset/mean-mails-pay.md

@@ -0,0 +1,5 @@
+---
+'@rocket.chat/meteor': patch
+---
+
+Fixes Slack messages being incorrectly saved on import

apps/meteor/app/importer/server/classes/converters/MessageConverter.ts

@@ -1,5 +1,6 @@
 import type { IImportMessageRecord, IMessage as IDBMessage, IImportMessage, IImportMessageReaction } from '@rocket.chat/core-typings';
 import { Rooms } from '@rocket.chat/models';
+import { removeEmpty } from '@rocket.chat/tools';
 import limax from 'limax';
 
 import type { UserIdentification, MentionedChannel } from './ConverterCache';
@@ -84,7 +85,7 @@ export class MessageConverter extends RecordConverter<IImportMessageRecord> {
 		const mentions = data.mentions && (await this.convertMessageMentions(data));
 		const channels = data.channels && (await this.convertMessageChannels(data));
 
-		return {
+		return removeEmpty({
 			rid,
 			u: {
 				_id: creator._id,
@@ -110,7 +111,7 @@ export class MessageConverter extends RecordConverter<IImportMessageRecord> {
 			alias: data.alias,
 			...(data._id ? { _id: data._id } : {}),
 			...(data.reactions ? { reactions: await this.convertMessageReactions(data.reactions) } : {}),
-		};
+		});
 	}
 
 	protected async convertMessageChannels(message: IImportMessage): Promise<MentionedChannel[] | undefined> {