-
Notifications
You must be signed in to change notification settings - Fork 76
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
fix types for channel.sendReaction #245
Conversation
types/stream-chat/index.d.ts
Outdated
@@ -113,6 +113,9 @@ export interface Reaction { | |||
[propName: string]: any; | |||
} | |||
|
|||
export interface ReactionRequest { | |||
type: string; |
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.
reactions have more fields, @bogdan-d please help here :)
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.
This is just for sending a reaction though. I was working on the typescript stuff and it was failing because we reused the Reaction response object for this.
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.
@jaapbakker88 Which properties of Reaction
were breaking? I can see message_id
, user
as required properties on Reaction
type. They are used for request using server side token. So maybe we can make them optional!!
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.
@vishalnarkhede updated to use only the type
as a required property. It was indeed breaking on the rest.
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.
Have one comment!!
@vishalnarkhede updated this |
Looks good now :) |
Submit a pull request
CLA
Description of the pull request