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

amp auto-ads not working #12614

Closed
kamurj opened this issue Dec 28, 2017 · 6 comments
Closed

amp auto-ads not working #12614

kamurj opened this issue Dec 28, 2017 · 6 comments

Comments

@kamurj
Copy link

kamurj commented Dec 28, 2017

Hello, I am having trouble finding the cause why adsense auto ads will not load on my amp pages. As far as I can tell the required code was added to the head and body section of the amp pages and while other ads load fine the auto ads don't.

Here are some example urls. Thank you for your help and feedback.

https://www.enotalone.com/forum/amp.php?t=539953
https://www.enotalone.com/forum/amp.php?t=544906

@lannka
Copy link
Contributor

lannka commented Dec 28, 2017

I got this in the error log:

amp-auto-ads config xhr failed: SyntaxError: Unexpected end of JSON input​​​

@tlong2 could you take a look?

@kamurj
Copy link
Author

kamurj commented Dec 28, 2017

Lannka I think that happens when viewed from a none mobile device?

@ampprojectbot ampprojectbot added this to the Backlog Bugs milestone Jan 1, 2018
@tlong2
Copy link
Contributor

tlong2 commented Jan 5, 2018

Hi,

AMP Auto Ads for AdSense is currently in a beta phase so the option to turn it on within the AdSense UI is not available to all AdSense accounts yet. If the client-side code is implemented without it being enabled in AdSense, then no configuration will be served and the error you see would occur.

We're working hard to make it available to everyone soon and I'll update this bug again once it is. Sorry for the inconvenience.

@kamurj
Copy link
Author

kamurj commented Jan 5, 2018

Thank you.

@jamescridland
Copy link

If I might make a suggestion - this bug looks to be the same as #16599.

Currently, ads do appear to show with amp-auto-ads on desktop, but there is a lot of complaining messages in the console about anchor elements not being found, and a failure to "fulfil ad strategy".

@lannka
Copy link
Contributor

lannka commented Jul 12, 2018

Closing this now, and we can further discuss the 2nd issue in #16599

@lannka lannka closed this as completed Jul 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
AMP Advertising
  
Awaiting triage
Development

No branches or pull requests

5 participants