[CHA-0] Add user_message_reminders field to ChannelType for channel type updates #219
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
CLA
Description of the pull request
Changes
user_message_remindersfield toChannelTypemodelChannelTypeCreateRequestData(for creating channel types)ChannelTypeUpdateRequestData(for updating channel types)Why
The
user_message_remindersflag was missing from the ChannelType model, which meant it was impossible to enable message reminders at the channel type level usingChannelType.update().While reminders could be enabled per-channel via
config_overrides, there was no way to set it as default for all channels of a specific type.JavaScript SDK Equivalent
This now allows the Java equivalent of:
Java Usage After Fix
Testing