Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release 1.8.1 #190

Merged
merged 13 commits into from
Dec 12, 2020
Merged

release 1.8.1 #190

merged 13 commits into from
Dec 12, 2020

Conversation

stklcode
Copy link
Contributor

@stklcode stklcode commented Dec 9, 2020

No description provided.

stklcode and others added 11 commits August 8, 2020 11:56
Release 1.8.0 back into develop
Standard and Transitional mode use the "amp_analytics_entries" hook,
Reader mode uses "amp_post_template_analytics". We only added a filter
for the latter, so AMP compatibility was only available in Reader mode.
Scripts and inline JS get removed on pages served via AMP. While that's
fine for Statify, because we do use a separate analytics hook, it raises
warnings for invalid markup on every AMP page. We now detect AMP usage
and skip embedding the script.

Also the checks are updated to use the new amp_is_request() function
introduced in AMP plugin 2.0 for future compatibility.
Commonly used methods "beacon" and "xhrpost" are enabled by default,
but as Statify doesn't work with "image", i.e. GET requests, we now
specify the supported methods explicitly again.
fix AMP compatibility for Standard and Transitional mode (#181)
Add integration tests for tracking, frontend, cron and dashboard widget

All initializations with custom options are triggered through the same
trait method to ensure complete options in every test case.

CI matrix with PHP Versions from 5.6 to 7.4, integrations tests against
oldest supported WP 4.7 with PHP 5.6 and latest WP 5.5 with PHP 7.4.

Co-authored-by: Florian Brinkmann <[email protected]>
Exclude sitemap and sitemap stylesheet calls with WP 5.5 core sitemaps
from tracking. Including unit tests.
@stklcode stklcode added this to the 1.8.1 milestone Dec 9, 2020
readme.txt Outdated Show resolved Hide resolved
Copy link
Member

@florianbrinkmann florianbrinkmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.
Made a suggestion for a small change in the changelog, but maybe it is just me and it is clear as it is :)

@florianbrinkmann
Copy link
Member

florianbrinkmann commented Dec 9, 2020 via email

@patrickrobrecht patrickrobrecht self-assigned this Dec 9, 2020
@patrickrobrecht patrickrobrecht merged commit 2ca4f51 into stable Dec 12, 2020
@patrickrobrecht patrickrobrecht deleted the release/1.8.1 branch December 12, 2020 11:54
@patrickrobrecht patrickrobrecht restored the release/1.8.1 branch December 12, 2020 12:02
@patrickrobrecht patrickrobrecht deleted the release/1.8.1 branch December 12, 2020 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants