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

VMAP ads stay muted on muted autoplay #4992

Closed
Andekas opened this issue Feb 10, 2023 · 3 comments · Fixed by #4995
Closed

VMAP ads stay muted on muted autoplay #4992

Andekas opened this issue Feb 10, 2023 · 3 comments · Fixed by #4995
Labels
component: ads The issue involves the Shaka Player ads API or the use of other ad SDKs priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@Andekas
Copy link

Andekas commented Feb 10, 2023

Have you read the FAQ and checked for duplicate open issues?

Yes

What version of Shaka Player are you using?

Latest

Can you reproduce the issue with our latest release version?

Yes

Can you reproduce the issue with the latest code from main?

Yes

Are you using the demo app or your own custom app?

Both

If custom app, can you reproduce the issue using our demo app?

What browser and OS are you using?

Windows, Chrome
For embedded devices (smart TVs, etc.), what model and firmware version are you using?

What are the manifest and license server URIs?

What configuration are you using? What is the output of player.getConfiguration()?

Standard configuration

What did you do?

Video is set to muted and autoplay. Ads are loaded through VMAP (client side ads).

For testing you can use any VMAP option from https://developers.google.com/interactive-media-ads/docs/sdks/html5/client-side/tags

What did you expect to happen?

Video starts muted and autoplays.
User unmutes video.
Video and ads play with sound.

What actually happened?

After unmuting, video plays with sound, but ads are still muted.

@Andekas Andekas added the type: bug Something isn't working correctly label Feb 10, 2023
@github-actions github-actions bot added this to the v4.4 milestone Feb 10, 2023
@Andekas
Copy link
Author

Andekas commented Feb 10, 2023

I did a quick fix, that resolves also #4800 issue:

adManager.addEventListener(shaka.ads.AdManager.IMA_AD_MANAGER_LOADED, (e) => {
  imaAdManager = e['imaAdManager'];
});

adManager.addEventListener(
  shaka.ads.AdManager.AD_STARTED, () => {
 imaAdManager.setVolume(!video.muted);
});

@avelad
Copy link
Member

avelad commented Feb 10, 2023

@Andekas are you interested on send a PR for it? Thanks!

@avelad avelad added component: ads The issue involves the Shaka Player ads API or the use of other ad SDKs priority: P2 Smaller impact or easy workaround labels Feb 10, 2023
@Andekas
Copy link
Author

Andekas commented Feb 13, 2023

@avelad I was just about to say that sure, but I guess I was too late for it. Thanks!

@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Apr 14, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: ads The issue involves the Shaka Player ads API or the use of other ad SDKs priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants