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

Automatically mark GA4 scripts as being PX-verified #7290

Merged
merged 36 commits into from
Oct 27, 2022

Conversation

thelovekesh
Copy link
Collaborator

@thelovekesh thelovekesh commented Oct 10, 2022

Summary

This PR adds a sanitizer(AMP_GTag_Script_Sanitizer) for GA4 scripts, marking GA4 scripts as px verified and hence users can use the latest GA while AMP implements it.

Apart from adding sanitizer, this PR update Option interface includes options used in Sandboxing service.

Fixes #7289

Checklist

  • My code is tested and passes existing tests.
  • My code follows the Engineering Guidelines (updates are often made to the guidelines, check it out periodically).

@thelovekesh thelovekesh marked this pull request as ready for review October 10, 2022 07:28
@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 2022

Plugin builds for aeacc33 are ready 🛎️!

Copy link
Member

@westonruter westonruter left a comment

Choose a reason for hiding this comment

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

Sorry for the delay with reviewing.

includes/amp-helper-functions.php Outdated Show resolved Hide resolved
includes/sanitizers/class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
src/Sandboxing.php Outdated Show resolved Hide resolved
tests/php/test-class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
tests/php/test-class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
tests/php/test-class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
tests/php/test-class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
tests/php/test-class-amp-gtag-script-sanitizer.php Outdated Show resolved Hide resolved
Co-authored-by: Weston Ruter <westonruter@google.com>
@westonruter westonruter added this to the v2.3.1 milestone Oct 27, 2022
@westonruter westonruter merged commit 86ad657 into develop Oct 27, 2022
@westonruter westonruter deleted the add/GA4-scripts-sanitizer branch October 27, 2022 17:46
@westonruter westonruter modified the milestones: v2.3.1, v2.4 Jan 26, 2023
@westonruter westonruter added the Changelogged Whether the issue/PR has been added to release notes. label Feb 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelogged Whether the issue/PR has been added to release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically mark GA4 scripts as being PX-verified
2 participants