ampproject / amphtml

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

amp-next-page: Google cache problem #36339

Closed spinmar closed 2 years ago

spinmar commented 2 years ago

Description

Hi, we are experimenting some problem about amp-next-page. We introduced it to solve CLS problem in our amp pages but it seems that when the page is served by Google cache the tag amp-next-page has some problem. The request in the network panel is present but the response is not displayed.

Reproduction Steps

To reproduce the problem follow the below instructions. Our amp page is this:

https://sport.virgilio.it/milan-in-emergenza-trattativa-lampo-per-il-sostituto-di-maignan-715489/amp/

it is a valid amp page with amp-next-page at the end. If you scroll at the end you can see that we are serving the footer with an ad. All is ok.

If you try to see tha page with Google AMP cache:

https://sport-virgilio-it.cdn.ampproject.org/c/s/sport.virgilio.it/milan-in-emergenza-trattativa-lampo-per-il-sostituto-di-maignan-715489/amp/

and you scroll at the end the footer is not served. By network panel you can see the request but it is not displayed.

Relevant Logs

No response

Browser(s) Affected

Chrome, Safari

OS(s) Affected

No response

Device(s) Affected

No response

AMP Version Affected

No response

dmanek commented 2 years ago

Hi @spinmar thanks for reporting this issue. After looking at both inks, I don't see the footer defined within amp-next-page. Can you add the footer attribute as described at https://amp.dev/documentation/components/amp-next-page/#footer-elements?

dmanek commented 2 years ago

Closing since there is no update.