better-studio / better-amp

Full Google AMP support for WordPress with custom themes and customizations
https://betterstudio.com/wp-plugins/better-amp/
GNU General Public License v2.0
19 stars 19 forks source link

Various issues #19

Open Lofesa opened 7 years ago

Lofesa commented 7 years ago

First, Thx for your work. Issues: 1.- I have a static page as is_front_page() page and other page as is_home() page.The plugin get the is_home() page as is_front_page() thus the amphtml and cononical tags are wrong. 2.- Links to categories in amp post get a 404 error 3.- Links in share buttons are not the pretty links but links like ?p=postID

Enhancements:

1.- You can consider to use the https://cdn.ampproject.org/v0/amp-user-notification-0.1.js to give a minimum compliance to cookies directives. 2.- Can consider to do SEO plugins (all most to the most used) compatibility to get structured data or create your own structured data set.

WP 4.7.1

harryqt commented 7 years ago

Links in share buttons are not the pretty links but links like ?p=postID, That's not in just AMP, they are in the main content too. I asked the support, they told me they did it on purpose.

Lofesa commented 7 years ago

Hi @aliaghdam Any plan to add structured data?

aliaghdam commented 7 years ago

Hi @Lofesa That is in v1.2 todo list.

Lofesa commented 7 years ago

Thx for your response. I have managed to include schema metadata from schema plugin by altering the amp.php file in the include directory.

Lofesa commented 7 years ago

I can not find any clues as to why the plugin confuses the front-page with the posts page. In customizer I desigated a page as front page and an other page as page for posts, but plugin convert the posts page into the front page so some things goes wrong as the rel canonical or the rel amphtml. When pages are paginated and in pages larger than 1, rel canonical don´t change to /page/X where X >= 2, rel canonical and rel amphtml it should change when page number change.

How I can get a separated front page and a page for posts?