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

fix(calling): Request mic (and camera) #1969

Merged
merged 2 commits into from
Apr 11, 2024
Merged

Conversation

Flemmli97
Copy link
Collaborator

What this PR does 📖

  • On MacOS calling was not working in prod due to no access to the microphone. This PR makes it so the app will request mic (and camera) access when needed

Which issue(s) this PR fixes 🔨

Additional comments 🎤

The prod build works on linux. The issue also mentions windows but i am unsure about that since idk if you need to request access to the mic or not there.
(though it can also be that the testing done in the issue was via a windows <-> mac communication which made it seem like so)

@github-actions github-actions bot added Don't merge yet DO NOT MERGE Missing dev review Still needs to be reviewed by a dev Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Apr 11, 2024
@dariusc93 dariusc93 added the Ready for testing Ready for QA to test label Apr 11, 2024
Copy link
Contributor

UI Automated Test Results Summary for MacOS/Windows

430 tests   369 ✅  2h 45m 32s ⏱️
 39 suites   59 💤
  3 files      1 ❌  1 🔥

For more details on these failures and errors, see this check.

Results for commit e291500.

Copy link
Contributor

UI Automated Tests execution is complete! You can find the test results report here

@stavares843 stavares843 added QA Tested QA has tested and approved and removed Ready for testing Ready for QA to test Missing dev review Still needs to be reviewed by a dev Don't merge yet DO NOT MERGE Failed Automated Test This PR is failing Luis's Appium test and needs revised labels Apr 11, 2024
@stavares843 stavares843 merged commit 3f2f5e8 into dev Apr 11, 2024
8 of 10 checks passed
@stavares843 stavares843 deleted the microphone_access_fix branch April 11, 2024 17:04
@github-actions github-actions bot removed the QA Tested QA has tested and approved label Apr 11, 2024
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

Successfully merging this pull request may close these issues.

bug(calling): Calling works on dev but not prod build
4 participants