search

ampproject / amphtml

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

AMP viewer sending wrong URL when infinite scroll is triggered? #27064

Closed AndrewKGuan closed 2 years ago

AndrewKGuan commented 4 years ago

Please see this video and watch the end of the URL changes: https://youtu.be/Clt6RMjopSI

The AMP viewer is not setting the URL correctly, instead it's setting the path to "/v/s/"

wassgha commented 4 years ago

Hey @AndrewKGuan ! Thanks for reporting, is this amp-next-page@1.0 or 0.1?

AndrewKGuan commented 4 years ago

Hi, thanks for taking a look! It's 0.1. Here's the page's url: https://www.forbes.com/sites/scottmendelson/2020/03/01/onward-pixar-my-hero-academia-impractical-jokers-wendy-weekend-box-office/amp/

wassgha commented 4 years ago

Do you mind checking if it's fixed in 1.0 (please test locally since the page won't be valid AMP yet)

AndrewKGuan commented 4 years ago

Sorry I'm not sure if I can do that. I can override the source code from forbes.com locally on my browser, but I'm not sure how I can put it inside a viewer to check.

nainar commented 4 years ago

Forbes should be using version 1.0 right now. Andrew can you have them check if they are still getting this issue.

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.