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

feat: including media permission #332

Conversation

MarceloZapatta
Copy link
Contributor

At linux was not possible to send audio via microphone or using the webcam, not sure but this was probably happening in others operating system.

image

This PR allow to set the permission for media (camera and microphone) per tab and now allowing to use camera or microphone.

image

@amanharwara
Copy link
Owner

LGTM, thanks for the contribution! Will create a new release with this soon.

@amanharwara amanharwara merged commit 7fe8f76 into amanharwara:master Oct 20, 2024
@@ -26,6 +27,7 @@ export const getDefaultTab = (): Tab => ({
config: {
theme: "dark",
notifications: true,
media: false,
Copy link
Owner

Choose a reason for hiding this comment

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

actually I think it makes sense to have this be on by default since most users will want it to work without having to turn on a setting.

i've already merged the PR, so I'll just make that change on main before I create the release

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree, it makes sense the default to be enabled!

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.

2 participants