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

Start now button abc tests #2265

Merged
merged 3 commits into from
Feb 16, 2016
Merged

Start now button abc tests #2265

merged 3 commits into from
Feb 16, 2016

Conversation

h-lame
Copy link
Contributor

@h-lame h-lame commented Feb 10, 2016

For: https://trello.com/c/GYqg2EEj/266-set-up-and-run-the-start-now-button-a-b-c-test

Set up ABC test for overseas-passport, calculate-your-child-maintenance, and marriage-abroad to try out different text on the "Start Now" button to see if that has any effect on the number of people that actually start the smart answer.

Complete and tested on integration. Ready to merge.

@floehopper
Copy link
Contributor

I've added the "Don't merge" label to indicate that this isn't ready to merge.

@h-lame
Copy link
Contributor Author

h-lame commented Feb 15, 2016

FYI: I've deployed this branch to integration so we can test the GA stuff. Once we're done, I'll put master back.

@chrisroos
Copy link
Contributor

@h-lame: Out of interest, do you know whether it would've been possible to deploy this to Heroku to test?

We want to run abc tests for three smart-answers on the text in the
"Start Now" button.  We haven't configured the multivariate test to talk
to GA correctly yet, as we're still working out the best way to collect
the results (custom dimension or content experiment).
@h-lame h-lame force-pushed the start-now-button-abc-tests branch from 6019af5 to 4dd4631 Compare February 16, 2016 10:15
@h-lame h-lame changed the title [WIP] Start now button abc tests Start now button abc tests Feb 16, 2016
@h-lame
Copy link
Contributor Author

h-lame commented Feb 16, 2016

@chrisroos TBH not sure, I think GA is only configured to work on certain domains.

That said, we tested it and saw data coming through to GA so this is now ready to merge. I've rebased against master (491aa07).

@floehopper
Copy link
Contributor

This looks good to me 👍 I'm happy for you to merge it.

h-lame added a commit that referenced this pull request Feb 16, 2016
@h-lame h-lame merged commit 1be8cce into master Feb 16, 2016
@h-lame h-lame deleted the start-now-button-abc-tests branch February 16, 2016 10:29
h-lame added a commit that referenced this pull request Feb 16, 2016
…ests"

This reverts commit 1be8cce, reversing
changes made to 491aa07.
@h-lame h-lame mentioned this pull request Feb 16, 2016
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.

3 participants