search

wp-pwa / wp-plugin

Old Frontity plugin. For the new one please visit https://github.com/frontity.
https://frontity.org
GNU General Public License v3.0
2 stars 1 forks source link

AMP urls should include the AMP server #78

Closed luisherranz closed 5 years ago

luisherranz commented 5 years ago

Right now, the AMP server is not included, and the amphtml link looks like: /abortion-rate-in-us-is-lowest-its-been-in-decades/?siteId=Yax2YDwgwGaeMmvwy&dev=false&type=post&id=363.

$ampServer is undefined in this point: https://github.com/frontity/wp-plugin/blob/dev/wp-pwa.php#L831

This looks like a bug introduced in 1.10 so I'd take a look at the way we are retrieving the new WP settings.

frontibotito commented 5 years ago

:tada: This issue has been resolved in version 1.10.2 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket: