Create missing AMP versions of pages - Githubissues

news-catalyst / tiny-news-demo

a demo of a tiny publishing system powered by gatsby + google docs

MIT License

1 stars 0 forks source link

Create missing AMP versions of pages #73

Closed jacqui closed 4 years ago

jacqui commented 4 years ago

While investigating issue #72 I noticed there were several AMP pages referenced in the gatsby build that didn't exist (AFAIK):

404 page http://tinynewsco.org/404.html/amp/
Topics index http://tinynewsco.org/topics/amp/
Each topic article index, ex: http://tinynewsco.org/topics/coronavirus/amp/
Subscribe http://tinynewsco.org/subscribe

There are likely others - these are the ones I found listed in the Netlify deploy log under "Insecure mixed content detected"

The topic index and per-topic article index pages are created in gatsby-node.js, at least, so that should be relatively straight forward to do.

jacqui commented 4 years ago

I'm about to open a PR that creates all of these missing pages - and fixes the html rendering issue in #93. I don't think we need to make an AMP version of the 404 page... but I'm willing to be convinced otherwise :)