Skip to content
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

Camera lights, leaving meetings, and console warnings #2794

Closed
samspycher opened this issue Oct 31, 2023 · 1 comment
Closed

Camera lights, leaving meetings, and console warnings #2794

samspycher opened this issue Oct 31, 2023 · 1 comment

Comments

@samspycher
Copy link

samspycher commented Oct 31, 2023

This line logs a console.warn message when an action occurs in a state that does not support a transition for the given action. It will log this warning if an Update happens in the Disconnecting state:

No transition found from Disconnecting with Update

I suspect this warning is occurring frequently in our case because of the way we leave meetings:

await  audioVideo.stopVideoInput();
await  audioVideo.stopAudioInput();
audioVideo.stop();

This feels paranoid, but I am doing it because the documented way to end meetings is apparently incomplete, given further documentation that seems justified given my experience and also several stackoverflow questions on the topic [1], [2] (bottom answer is where I take my code from).

I have two questions:

  • Is the WARN level here correct? I can imagine situations where actions may happen concurrent with states for which there is no defined transition, and that being a benign situation.
  • What is the one true way to fully leave a meeting client-side in the way users would expect, without that logic creating console.warn messages in our logs?

Note: From my perspective as a user of this API , I do think fully leaving meetings (and releasing all employed resources) should be a single line of code. Perhaps I am missing something? But if not, the percentage of users who are googling in frustration after chapter 4 left their camera light running has got to be close to 100%.

@samspycher
Copy link
Author

Closing this, because I didn't observe the issue template. Apologies!

New issue is #2796.

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

No branches or pull requests

1 participant