Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix ad request muted state #427

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

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

vs-gianni
Copy link
Contributor

The ad request currently ignores if the player is in muted state, accounting only if the player can autostart with audio.
This will cause policy violations, because the ad request does not reflect the real audibility of the ad placement.

This fix addresses the problem, by taking into account the actual muted state of the player.

The ad request ignores if the player is in muted state, accounting only if the player can autostart with audio.
This will cause policy violations, because the ad request will not reflect the real audibility of the ad placement.

This fix addresses the problem, by taking into account the actual muted state of the player.
@rafa8626 rafa8626 merged commit a53e3e5 into openplayerjs:master Oct 7, 2024
2 of 3 checks passed
@vs-gianni vs-gianni deleted the fix_ad_request_muted_state branch October 7, 2024 19:34
@vs-gianni vs-gianni restored the fix_ad_request_muted_state branch October 7, 2024 19:35
@vs-gianni
Copy link
Contributor Author

thank you @rafa8626
is there a new release planned anytime soon?

@rafa8626
Copy link
Member

rafa8626 commented Oct 7, 2024

@vs-gianni I’ll release the new version later tonight. If you haven’t star the project, could you give it a star to show your support please?

@vs-gianni
Copy link
Contributor Author

Sure, it's a great project: thank you for your commitment!

@rafa8626
Copy link
Member

rafa8626 commented Oct 9, 2024

@vs-gianni I had some urgent thing for work, so I’ll try to publish the version tomorrow first thing

@vs-gianni
Copy link
Contributor Author

Thank you for the update @rafa8626

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