-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
chore(ui-contexts): useGoToRoom
#32817
Conversation
|
Looks like this PR is ready to merge! 🎉 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #32817 +/- ##
===========================================
+ Coverage 55.57% 55.60% +0.03%
===========================================
Files 2634 2634
Lines 57279 57252 -27
Branches 11860 11853 -7
===========================================
+ Hits 31832 31836 +4
+ Misses 22791 22760 -31
Partials 2656 2656
Flags with carried forward coverage won't be shown. Click here to find out more. |
9c80057
to
58f79df
Compare
1d7f083
to
5b4ef26
Compare
… name instead of the expected id
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.
LGTM!
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments