ampproject / amphtml

The AMP web component framework.
https://amp.dev
Apache License 2.0
14.89k stars 3.89k forks source link

AMP ads on NON-AMP pages #28969

Closed aviz1 closed 4 years ago

aviz1 commented 4 years ago

Please only file reports about bugs in AMP here.

If you have a bug for AMP please fill in the following template. Delete everything except the headers (including this text).

What's the issue?

You didn't answer my last question and closed the issue !!!! Please check it https://github.com/ampproject/amphtml/issues/28878#issuecomment-645771012

Briefly describe the bug.

How do we reproduce the issue?

Please provide a public URL and ideally a reduced test case (e.g. on jsbin.com) that exhibits only your issue and nothing else. Provide step-by-step instructions for reproducing the issue:

  1. Step 1 to reproduce
  2. Step 2 to reproduce

What browsers are affected?

All browsers? Some specific browser? What device type?

Which AMP version is affected?

Is this a new issue? Or was it always broken? Paste the version of AMP where you saw this issue. (You can find the version printed in your browser's console.)

Ichigo3d commented 4 years ago

Sorry but you can't add amp-ads to Not AMP pages. In screenshot which you provided, you was little bit mixed up: "In non-AMP pages (traditional HTML), if you want to display ads on your page, you'd include a snippet of JavaScript to serve ads from your ad network. " - it's said about Not AMP ADS on Not AMP pages. And in next sentence, it's said about how to better and faster use AMP ADs in AMP pages. Sorry for your confusing, we will modify this page for avoid confuses in the future. Thank you.

Denend commented 3 years ago

@Ichigo3d @aviz1 However it is possible to serve amp ads on non-amp pages, probably not with amp-ad component but still. See the implementation of the ads on https://www.gismeteo.ua