search

juusaw / amp-jekyll

Build Accelerated Mobile Page versions of your Jekyll posts
MIT License
281 stars 59 forks source link

Amp generation doesn't respect the permalink configuration #29

Open vndmtrx opened 7 years ago

vndmtrx commented 7 years ago

I noticed that when a different permalink configuration is used, the generation of the amp page does not follow the same pattern. I believe this could have relation with #27.

For example, I am attaching the configuration files of a project in which the generated link should be /2017/05/15/welcome-to-jekyll.html and the plugin is generating /amp/2017/05/15/welcome-to-jekyll/ for the amp page, when they should generate a /amp/2017/05/15/welcome-to-jekyll.html page.

All the rest is the same from jekyll new site plus the configs the project's README.md.

layout_teste.zip

vndmtrx commented 7 years ago

Inspecting the code, I notice that amp-jekyll uses site.pages to store the newly generated amp pages, and these pages follow the pages url pattern.

Looking futher, I came with a conclusion that to make this work in respect of URL pattern from original post, it need changes on the way Jekyll builds pages and posts.

I don't know how this could be achieved here, but I let this for future users that encounter the same problem as I.