-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
feat(rn) start listening for stats on the Thumbnail #10583
base: master
Are you sure you want to change the base?
feat(rn) start listening for stats on the Thumbnail #10583
Conversation
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.
Left some comments, PTAL. Also, the linter dislikes you :-P
react/features/video-menu/components/native/ConnectionStatusComponent.js
Show resolved
Hide resolved
react/features/video-menu/components/native/ConnectionStatusComponent.js
Outdated
Show resolved
Hide resolved
0a6953b
to
0698107
Compare
0698107
to
af39a84
Compare
af39a84
to
cfc4efb
Compare
I gave this a quick test and I don't see stats for the local participant (simulator, but I should see them I believe) and for the remote, once I get them all, if I close and open the menu half of them go N/A and they come back after a bit. Can you PTAL? |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
@Calinteodor Let's try to push this over the finish line when time allows. |
No description provided.