Skip to content

Conversation

@sasha1107
Copy link

@sasha1107 sasha1107 commented Jan 2, 2025

The details of the pull request I am proposing involve using messageSearchParams to provide to the messageSearchProvider. Currently, there is no way to input a messageSearchQuery without a searchString.

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If unsure, ask the members.
This is a reminder of what we look for before merging your code.

  • All tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • Public components / utils / props are appropriately exported
  • I have added necessary documentation (if appropriate)

@netlify
Copy link

netlify bot commented Jan 2, 2025

👷 Deploy request for sendbird-uikit-react pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 2d22e0f

@sasha1107 sasha1107 marked this pull request as ready for review January 2, 2025 05:33
@HoonBaek
Copy link
Contributor

HoonBaek commented Jan 8, 2025

Thank you for your interest in our library. @sasha1107
We will test whether this feature is suitable and respond regarding the application of the proposed changes.

@HoonBaek
Copy link
Contributor

#1300

@HoonBaek HoonBaek closed this Jan 14, 2025
HoonBaek added a commit that referenced this pull request Jan 23, 2025
Made from #1292
Thanks to @sasha1107

### Changelog
* Also use the `messageSearchQuery.keyword` when searching in
MessageSearch module
* Note: The `searchString` has higher priority, P0: `searchString`, P1:
`messageSearchQuery.keyword`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants