-
Notifications
You must be signed in to change notification settings - Fork 77
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(search): missing thread_participants in message #1412
Conversation
Size Change: 0 B Total Size: 461 kB ℹ️ View Unchanged
|
@@ -1713,6 +1713,10 @@ export type MessageFilters<StreamChatGenerics extends ExtendableGenerics = Defau | |||
} | |||
>; | |||
|
|||
export type MessageOptions = { |
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.
I think that only MessageOptions
would not be clear enough to convey the fact that these are actually search payload options.
export type MessageOptions = { | |
export type SearchMessageOptions = { |
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.
Yeah I thought about it as well, however, technically, in the future, it could be used in other places where the MessageFilters
is also used.
6c9d0dc
to
7102756
Compare
Link to issue
Related PR
Summary
Enable the
Channel.search
to enrichthread_participants
in found messages.CLA
Changelog