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.

Sign up for GitHub

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

Jump to bottom

PLANET-7676 Added snippet for VWO Anti Flickering #2487

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

PLANET-7676 Added snippet for VWO Anti Flickering #2487

wants to merge 1 commit into from

Conversation

Osong-Michael
Copy link
Contributor

@Osong-Michael Osong-Michael commented Dec 19, 2024 •
edited
Loading

Summary

Ref: https://jira.greenpeace.org/browse/PLANET-7676

Testing

  1. When the VWO A/B testing option is enabled in Planet4 > Analytics the VWO script to prevent flickering during testing is added to the site. You can inspect the head tag to find the script.
  2. You can check the international dev-instance, when the Option is enabled, the script is added to the head tag.

@Osong-Michael Osong-Michael self-assigned this Dec 19, 2024
planet-4 added a commit to greenpeace/planet4-test-telesto that referenced this pull request Dec 19, 2024
@planet-4
greenpeace/planet4-master-theme#2487 at 96c09b3a
ba1cc14 
/unhold 1af8d327-ea0c-44da-959a-042f117ed36c
@planet-4
Copy link
Contributor

planet-4 commented Dec 19, 2024 •
edited
Loading

Test instance is ready 🚀

🌑 telesto | admin | blocks report | CircleCI | composer-local.json

⌚ 2025.01.22 12:50:23

@Osong-Michael Osong-Michael added Review UAT Needed This PR requires User Acceptance Tests before merge and removed WIP [Test Env] telesto labels Dec 19, 2024
sagarsdeshmukh
sagarsdeshmukh approved these changes Jan 21, 2025
View reviewed changes
Copy link
Member

@sagarsdeshmukh sagarsdeshmukh left a comment

Choose a reason for hiding this comment

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

PR looks good to me 👍

@Osong-Michael
PLANET-7676 Added snippet for VWO Anti Flickering
9510438 
- Added code to prevent flickering when A/B testing using VWO
planet-4 added a commit to greenpeace/planet4-test-telesto that referenced this pull request Jan 22, 2025
@planet-4
greenpeace/planet4-master-theme#2487 at 95104388
95ebf37 
/unhold 426cfd02-ee1f-44ba-a086-c41d9255fa37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sagarsdeshmukh sagarsdeshmukh sagarsdeshmukh approved these changes

@GP-Dan-Tovbein GP-Dan-Tovbein Awaiting requested review from GP-Dan-Tovbein

Assignees

@Osong-Michael Osong-Michael

Labels
Review UAT Needed This PR requires User Acceptance Tests before merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Osong-Michael @planet-4 @sagarsdeshmukh