search

pluginkollektiv / statify

Statify – statistics plugin for WordPress
https://wordpress.org/plugins/statify/
GNU General Public License v3.0
76 stars 22 forks source link

add "?amp" instead of "/amp/" for AMP visit tracking (#183) #195

Open stklcode opened 3 years ago

stklcode commented 3 years ago

This is one possible way to fix #183

Instead of adding /amp/ to the URL of the page is accessed via AMP, we now add ?amp . This is always fine, because AMP should never provide other URL parameters.

Depending on the outcome of the discussion in #183 we can either merge this PR or close it. Just opened, so we don't do the work twice if needed.

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication