chatkit-server-node icon indicating copy to clipboard operation
chatkit-server-node copied to clipboard

sendSimpleMessage should accept SendSimpleMessageOptions

Open zmarkan opened this issue 6 years ago • 0 comments

What?

At the moment sendSimpleMessage accepts SendMessageOptions, which takes an optional attachment: AttachmentOptions property.

This is incorrect for sending simple, text-only messages, and will appear as a suggestion in VSCode and other IDEs that support TS typings.

Suggested improvements

Create a new interface SendSimpleMessageOptions that doesn't include the attachment property.

zmarkan avatar Jul 02 '19 11:07 zmarkan