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

Build a sandbox for CRO #7508

Closed
1 task
hoosteeno opened this issue Jul 31, 2019 · 3 comments
Closed
1 task

Build a sandbox for CRO #7508

hoosteeno opened this issue Jul 31, 2019 · 3 comments
Assignees
Labels
P1 First level priority - Must have

Comments

@hoosteeno
Copy link
Contributor

hoosteeno commented Jul 31, 2019

Description

Implement the sandbox defined in #7505: https://docs.google.com/document/d/1RXzBVUk3LSbyBeJOesPrKcekvA5bOUBiUIiQy7frHgQ/edit?usp=sharing


💛 Success Criteria 💛

  • Sandbox is built and a) ready to port content to b) ready to send audience to
@hoosteeno hoosteeno added this to the S3/Q3: 20190812-20190830 milestone Jul 31, 2019
@hoosteeno hoosteeno added the P1 First level priority - Must have label Aug 9, 2019
@alexgibson alexgibson self-assigned this Sep 2, 2019
@alexgibson
Copy link
Member

The initial pages in the sandbox will be:

  • /en-US/exp/firefox/new/
  • /de/exp/firefox/new/

For now we'll be hardcoding the strings for de, rather than exposing the strings through pontoon.

@alexgibson
Copy link
Member

alexgibson commented Sep 11, 2019

Depends on #7700

@alexgibson
Copy link
Member

@hoosteeno I've implemented convert in the PR referenced above, and pushed it to demo here:

https://www-demo3.allizom.org/en-US/exp/firefox/new/

In the convert dashboard I've set up a simple A/B test (in the staging project) that changes the headline. It seems to be working ok and tracking as expected. Please take a look.

Things to note:

  • I've implemented loading their script dynamically (async). This means that if the script is slow to load, or unresponsive, our visitors won't be left looking at a blank screen & loading spinner. This can mean that swapping out varation content may be visible during page load, but as long as their CDN is fast I don't think the experience is too jarring. This is a better (imho) alternative to blocking on a third-party resource.

alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 11, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 11, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 12, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 13, 2019
@alexgibson alexgibson added the Needs Review Awaiting code review label Sep 13, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 17, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 18, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 18, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 18, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 18, 2019
alexgibson pushed a commit to alexgibson/bedrock that referenced this issue Sep 19, 2019
@stephaniehobson stephaniehobson removed the Needs Review Awaiting code review label Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 First level priority - Must have
Projects
None yet
Development

No branches or pull requests

4 participants