Signup: suspend Site Title AB test #10357

Merged
merged 1 commit into from Jan 2, 2017

Projects

None yet

5 participants

@michaeldcain
Member
michaeldcain commented Jan 1, 2017 edited

Now that we've let the Site Title test run at 50/50, we're suspending the test until we make a decision about its future.

To test:

  1. Checkout branch or use Calypso.live link
  2. Start signup
  3. Verify there are no JS errors
  4. Verify Signup completes without the Site Title step.
@michaeldcain michaeldcain added the Signup label Jan 1, 2017
@bisko

The test date needs a change.

client/lib/abtest/active-tests.js
@@ -81,10 +81,10 @@ module.exports = {
allowExistingUsers: true,
},
siteTitleStep: {
- datestamp: '20160928',
+ datestamp: '20180101',
@bisko
bisko Jan 1, 2017 Contributor

Back to the future :)

@marekhrabe
Contributor

Except for the date string, it looks good I think. I signed up - no js errors regarding this patch, no site title step seen. 🚢

@bisko
bisko approved these changes Jan 1, 2017 View changes

LGTM! 👍

@michaeldcain michaeldcain Signup: suspend Site Title AB test
91766df
@michaeldcain michaeldcain merged commit b109362 into master Jan 2, 2017

2 checks passed

ci/circleci Your tests passed on CircleCI!
Details
ci/i18n 0 new strings. ¡Ándale!
@lancewillett lancewillett deleted the update/signup-site-title-ab branch Jan 3, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment