-
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
Create fields for calendar and messaging settings v2 #6049
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 new standard field IDs for calendar and messaging settings
- Introduced
contactAutoCreation
field toCalendarChannelWorkspaceEntity
- Added
contactAutoCreation
,excludeNonProfessionalEmails
, andexcludeGroupEmails
fields toMessageChannelWorkspaceEntity
- Deprecated
isContactAutoCreationEnabled
field in messaging settings - Updated enums for contact auto-creation and sync stages in messaging settings
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
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.
Left comments about the naming!
...ges/twenty-server/src/modules/calendar/standard-objects/calendar-channel.workspace-entity.ts
Outdated
Show resolved
Hide resolved
...nty-server/src/modules/messaging/common/standard-objects/message-channel.workspace-entity.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)
- Updated default value for
contactAutoCreation
incalendar-channel.workspace-entity.ts
- Ensured
contactAutoCreation
andsyncStage
defaults are strings inmessage-channel.workspace-entity.ts
2 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
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)
- Renamed
contactAutoCreation
tocontactAutoCreationPolicy
instandard-field-ids.ts
- Updated
CalendarChannelContactAutoCreation
toCalendarChannelContactAutoCreationPolicy
incalendar-channel.workspace-entity.ts
- Renamed
MessageChannelContactAutoCreation
toMessageChannelContactAutoCreationPolicy
inmessage-channel.workspace-entity.ts
- Ensured consistency and clarity in naming conventions for contact auto-creation policies
3 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings
Create fields for calendar and messaging settings v2