-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
5901 refactor email and calendar auto contact creation to create them by batch #6038
5901 refactor email and calendar auto contact creation to create them by batch #6038
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
- Added
@types/lodash.chunk
andlodash.chunk
dependencies - Introduced
CONTACTS_CREATION_BATCH_SIZE
constant - Deleted
messaging-message-channel.listener.ts
- Refactored
create-company-and-contact.service.ts
to handle batch processing - Updated type definitions from
Contacts
toContact[]
across multiple files
9 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings
objectRecordChangedProperties( | ||
payload.properties.before, | ||
payload.properties.after, | ||
).includes('isContactAutoCreationEnabled') && | ||
payload.properties.after.isContactAutoCreationEnabled |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🪶 style: Consider adding error handling for the case where 'objectRecordChangedProperties' fails or returns unexpected results.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
...saging/message-import-manager/listeners/messaging-import-manager-message-channel.listener.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Added
AutoCompaniesAndContactsCreationMessageChannelListener
to providers - Introduced listener for
messageChannel.updated
event - Batched creation of companies and contacts for efficiency
- Removed
messageChannel.updated
listener frommessaging-import-manager-message-channel.listener.ts
- Focused
messaging-import-manager-message-channel.listener.ts
onmessageChannel.deleted
event for cache cleaning
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
…t-creation-to-create-them-by-batch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR Summary
(updates since last review)
- Refactored email and calendar auto-contact creation to handle batch processing.
- Replaced
useSetRecordInStore
withuseUpsertRecordsInStore
across multiple components for better record management. - Introduced new hooks and components for multi-record selection and handling.
- Added
WorkspaceJoinColumn
decorators to various entity fields for improved join column handling. - Updated GraphQL queries and mutations to support batch operations and upsert functionality.
215 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
Closes #5901