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

IE doesn't set the subscriber's volume #27

Open
ulifigueroa opened this issue Jan 2, 2017 · 5 comments
Open

IE doesn't set the subscriber's volume #27

ulifigueroa opened this issue Jan 2, 2017 · 5 comments

Comments

@ulifigueroa
Copy link

Even here the volume is specified IE 11 doesn't change the volume and the user can her herself while doing the network test. https://github.com/opentok/opentok-network-test/blob/master/js-sample/app.js#L100

I also tried doing subscriber.setAudioVolume(0); and it doesn't' work? Is this a Tokbox plugin issue? Is there any workaround from the Javascript side?

@ulifigueroa ulifigueroa changed the title IE doesn't set the subscriber volume IE doesn't set the subscriber's volume Jan 2, 2017
@aullman
Copy link

aullman commented Jul 28, 2017

Hm, can we use subscriber.subscribeToAudio(false) instead?

@davemun
Copy link
Contributor

davemun commented Jul 28, 2017

subscriber.subscribeToAudio(false) in the session.subscribe() callback does work as a workaround at the moment.

I have nevertheless filed a bug internally for the setAudioVolume(0) not working issue.

@ulifigueroa
Copy link
Author

If I do that the network test will be running only for video. The drawback would be that the full bandwidth test (Audio and Video) cannot be done.

@davemun
Copy link
Contributor

davemun commented Jul 28, 2017

Ah, sorry, you are correct. I was just thinking about the volume part. subscribeToAudio(false) would not work for the original point of the test.

@ulifigueroa
Copy link
Author

Yes, for muting the audio purposes I agree that's the correct solution but in this case it doesn't work. So right now users are hearing themselves during the system test, which is kinda awkward.

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