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

Have to allow mic permissions for each video page #27

Open
jasonnelson1990 opened this issue Mar 19, 2014 · 10 comments
Open

Have to allow mic permissions for each video page #27

jasonnelson1990 opened this issue Mar 19, 2014 · 10 comments

Comments

@jasonnelson1990
Copy link
Collaborator

No description provided.

@jasonnelson1990
Copy link
Collaborator Author

Appears that this is a browser specific problem. One solution is to include documentation on how to set browser settings to avoid this problem. Not sure if there's an app-solution to this problem.

@ke-nw
Copy link
Collaborator

ke-nw commented Mar 19, 2014

There's nothing we can do in-app for this. Each browser lets you make exceptions that grant mic permissions to an entire website. For Chrome, see: https://support.google.com/chrome/answer/2693767?p=ib_access_cam_mic&rd=1

@jasonnelson1990
Copy link
Collaborator Author

It's under the advanced settings, manage exceptions hyperlink which leads to another help page. I sent it to our sponsor with some comments, my hope is she can get it working.
In one sense we could just tell her subjects "go into your browser settings", but it might be worth writing some quick set-up guides for each expected browser, so it's all in one place and as compact as possible. But you're right, this isn't actually a problem with our app.

@ke-nw
Copy link
Collaborator

ke-nw commented Mar 19, 2014

The other thing to consider is that most people don't WANT to grant blanket
permissions to an application to allow it to access their microphone at any
time, for any reason...

On Wed, Mar 19, 2014 at 1:58 PM, Jason Nelson notifications@github.comwrote:

It's under the advanced settings, manage exceptions hyperlink which leads
to another help page. I sent it to our sponsor with some comments, my hope
is she can get it working.
In one sense we could just tell her subjects "go into your browser
settings", but it might be worth writing some quick set-up guides for each
expected browser, so it's all in one place and as compact as possible. But
you're right, this isn't actually a problem with our app.

Reply to this email directly or view it on GitHubhttps://github.com//issues/27#issuecomment-38105878
.

@hambone20
Copy link
Collaborator

It may be possible to refactor the mic code so it only asks for permission once per run. I think it will involve some J query finagling, but I can look at it tomorrow.

@jasonnelson1990
Copy link
Collaborator Author

That's essentially what the problem is. Accepting it once might not be so bad, but it asks for it every time a video page is loaded, and our sponsor feels this ruins the simulation part of the simulated conversation.

@hambone20
Copy link
Collaborator

I have it working so it only asks you once per conversation. I need to do some cleanup before I push it but it seems like this is the best fix for this issue.

@jasonnelson1990
Copy link
Collaborator Author

Great! I'll pull it after you push it, test it, and we'll see about getting it on ARC.

Date: Thu, 20 Mar 2014 12:45:10 -0700
From: notifications@github.com
To: Simulated-Conversations@noreply.github.com
CC: jasonnelson1990@hotmail.com
Subject: Re: [Simulated-Conversations] Have to allow mic permissions for each video page (#27)

I have it working so it only asks you once per conversation. I need to do some cleanup before I push it but it seems like this is the best fix for this issue.


Reply to this email directly or view it on GitHub.

@hambone20
Copy link
Collaborator

Ok, it's pushed, still needs some general cleanup, but to a user it should work as before, but without the constant loading of the recorder

@hambone20 hambone20 reopened this Mar 20, 2014
@jasonnelson1990
Copy link
Collaborator Author

I just tried it out on Firefox and on OperaNext, and I only got the message the first time. I'm prepared to say this issue is resolved, pending code cleanup.

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

3 participants