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

Skip echo test in breakout room #11381

Merged
merged 5 commits into from Feb 11, 2021
Merged

Conversation

ramonlsouza
Copy link
Member

@ramonlsouza ramonlsouza commented Feb 11, 2021

What does this PR do?

Includes PR #8696 and resolves conflicts

Further information

If user has disabled cookies or does not join audio in main room, it will be prompted to do an echo test (default settings skipCheck: false and autoJoin: true). If skipCheck: true, it will auto join audio the same way as in main room.

@antobinary antobinary added this to the Release 2.3 milestone Feb 11, 2021
@antobinary antobinary merged commit 81f5302 into bigbluebutton:develop Feb 11, 2021
@maths22
Copy link
Contributor

maths22 commented Feb 21, 2021

This PR accidentally broke bbb_skip_check_audio_on_first_join. In particular the changes to bigbluebutton-html5/imports/ui/components/audio/audio-modal/container.jsx combined with the version of joinMicrophone in bigbluebutton-html5/imports/ui/components/audio/audio-modal/service.js dropped the check for skipCheckOnJoin

@antobinary
Copy link
Member

@maths22 thanks for the note!
@ramonlsouza could you please try to reproduce?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants