frontend
frontend copied to clipboard
Use www. url in `linkedData` when rendering non amp pages
What does this change?
Fixes https://github.com/guardian/frontend/issues/25262
Changes the @id of the Schema JSON from https://amp.theguardian.com/{id} to https://www.theguardian.com/{id} when we are not on AMP pages.
This is how it was represented on frontend, and is also best practise.
Screenshots
Frontend
URL: https://www.theguardian.com/politics/2022/sep/21/liz-truss-dismisses-putins-nuclear-threats-as-statement-of-weakness?dcr=false
Before

After

@OllysCoding
whether it would impact the discoverability of AMP pages?
I think you're right in the sense of that AMP articles should use the prop explicitly meant to expose AMP articles. If this were to affect that, would you expect AMP articles to stop being exposed in Google (just trying to think of a test)?
Marking as draft while we think about how we might validate this doesn't affect things negatively.
Seen on PROD (merged by @jamesgorrie 15 minutes and 54 seconds ago)
- Check your changes on www.theguardian.com ✔️
- Keep an eye on the deploy dashboard 📉