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: make setChannels prop optional for ChannelSearch #2358
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2358 +/- ##
=======================================
Coverage 85.23% 85.23%
=======================================
Files 353 353
Lines 8051 8051
Branches 2123 2123
=======================================
Hits 6862 6862
Misses 1005 1005
Partials 184 184 ☔ View full report in Codecov by Sentry. |
Size Change: +30 B (0%) Total Size: 1.82 MB
ℹ️ View Unchanged
|
@myandrienko why is it important to make this optional for the hook. Do we want to make this hook publicly available? |
@MartinCupela the hook Currently, if you want to use |
🎉 This PR is included in version 11.15.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
🎯 Goal
setChannels
is used by theChannelSearch
component to update the channel list state to include the selected search result. When rendered by theChannelList
, it actually passes its ownsetChannels
setter there.But this behavior should be optional, because it's totally valid to use search outside of the
ChannelList
.