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

Ads dont play before the video #105

Closed
aakashr143 opened this issue Feb 2, 2021 · 7 comments
Closed

Ads dont play before the video #105

aakashr143 opened this issue Feb 2, 2021 · 7 comments

Comments

@aakashr143
Copy link

On both iOS and android ads dont play before the video

  • iphone 11 Pro Max running iOS 13
  • iphone 5 S running iOS 12
  • Nokia 3.4 running android 10
@LonelyCpp
Copy link
Owner

LonelyCpp commented Feb 2, 2021

since there is no domain for the iframe to verify, youtube will not play ads. (#98 too is a related issue)

p.s: how youtube decides to show ads

@prisar
Copy link

prisar commented Feb 24, 2021

Is it ok if ads are not displayed. will it be in compliance of YouTube policy

@LonelyCpp
Copy link
Owner

@prisar definitely not a violation of policy. The library itself does nothing to prevent ads, youtube decides this based on the origin URL

@prisar
Copy link

prisar commented Feb 25, 2021

@LonelyCpp My app just got removed from the store for being non-compliant in Youtube policies. Earlier I got a review rejected for playing the video in background. Which actually was not happening. I am not sure why, thought that it maybe related to displaying ads. Currently, there are not ads in the app.

@LonelyCpp
Copy link
Owner

it's unlikely to be because of ads. The youtube native sdk (provided by google) also does not show ads.

The background play might be the cause since google has some automated tests to check this. To be 100% sure, use AppState to only render the component if currentState is active.

@LonelyCpp
Copy link
Owner

here's another related thread - #72

@prisar
Copy link

prisar commented Mar 9, 2021

@LonelyCpp After putting the condition for currentState to be inactive and background seems to be compliant. #72 has the same issue from what I have read. It really helped.
Now the app is available again:)

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