Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

Conversation

@viniciuslrangel
Copy link
Contributor

@viniciuslrangel viniciuslrangel commented Oct 10, 2018

Option to disable audio
Avoids requesting unnecessary permissions, especially when using the camera only for photos
Android implementation only

Option to disable audio
Avoids requesting unnecessary permissions, especially when using the camera only for photos
@collinjackson
Copy link
Contributor

possibly redundant with #904

@matin
Copy link

matin commented Dec 26, 2018

What's the status of this PR?

@viniciuslrangel
Copy link
Contributor Author

What's the status of this PR?

I didn't test on iOS, but I'm using on Android for a while and works great.

@bparrishMines
Copy link
Contributor

Hi @viniciuslrangel,

Thanks for the contribution! I'm going to take over the PR and try and get the iOS side working.

@bparrishMines bparrishMines changed the title [camera] Audio optional [camera] Add the ability to disable audio for video recording and image streaming May 16, 2019
Copy link
Contributor

@bparrishMines bparrishMines left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for Android portion

@mklim can you review the iOS portion since I'm the one that added it?

Copy link
Contributor

@mklim mklim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants