admin/twenty
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Removing empty handles from MessageWithParticipants (#10625)

Browse Source 
fixes https://github.com/twentyhq/twenty/issues/10623
This commit is contained in:
Guillim
2025-03-04 14:53:24 +01:00
committed by GitHub
parent 150e6bb17a
commit 696c510933
2 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
packages/twenty-server/src/modules/messaging/message-import-manager/services/messaging-save-messages-and-enqueue-contact-creation.service.ts
View File

@ -83,6 +83,7 @@ export class MessagingSaveMessagesAndEnqueueContactCreationService {
isGroupEmail(participant.handle); isGroupEmail(participant.handle);
const shouldCreateContact = const shouldCreateContact =
!!participant.handle &&
!isParticipantConnectedAccount && !isParticipantConnectedAccount &&
!isExcludedByNonProfessionalEmails && !isExcludedByNonProfessionalEmails &&
!isExcludedByGroupEmails && !isExcludedByGroupEmails &&

10
packages/twenty-server/src/modules/messaging/message-import-manager/utils/filter-emails.util.ts
View File

@ -66,10 +66,12 @@ const filterOutInternals = (
} }
const primaryHandleDomain = getDomainNameByEmail(primaryHandle); const primaryHandleDomain = getDomainNameByEmail(primaryHandle);
const isAllHandlesFromSameDomain = message.participants.every( const isAllHandlesFromSameDomain = message.participants
(participant) => .filter((participant) => !!participant.handle)
getDomainNameByEmail(participant.handle) === primaryHandleDomain, .every(
); (participant) =>
getDomainNameByEmail(participant.handle) === primaryHandleDomain,
);
if (isAllHandlesFromSameDomain) { if (isAllHandlesFromSameDomain) {
return false; return false;