diff --git a/services/web/app/coffee/Features/Subscription/SubscriptionController.coffee b/services/web/app/coffee/Features/Subscription/SubscriptionController.coffee index 03e87125ac..c89545ac68 100644 --- a/services/web/app/coffee/Features/Subscription/SubscriptionController.coffee +++ b/services/web/app/coffee/Features/Subscription/SubscriptionController.coffee @@ -30,7 +30,7 @@ module.exports = SubscriptionController = plans: plans gaExperiments: Settings.gaExperiments.plansPage recomendedCurrency:recomendedCurrency - shouldABTestPlans: currentUser == null or (currentUser?.signUpDate? and currentUser.signUpDate >= (new Date('2016-10-27'))) + shouldABTestPlans: currentUser == null or (currentUser?.signUpDate? and currentUser.signUpDate >= '2018-06-06') planFeatures: planFeatures user_id = AuthenticationController.getLoggedInUserId(req) if user_id? diff --git a/services/web/public/coffee/ide.coffee b/services/web/public/coffee/ide.coffee index 6c5b1d920e..88619aeb09 100644 --- a/services/web/public/coffee/ide.coffee +++ b/services/web/public/coffee/ide.coffee @@ -88,7 +88,7 @@ define [ $scope.user = window.user $scope.shouldABTestPlans = false - if $scope.user.signUpDate >= '2016-10-27' + if $scope.user.signUpDate >= '2018-06-06' $scope.shouldABTestPlans = true $scope.settings = window.userSettings diff --git a/services/web/test/unit/coffee/Subscription/SubscriptionControllerTests.coffee b/services/web/test/unit/coffee/Subscription/SubscriptionControllerTests.coffee index 1592215040..0d1929d46a 100644 --- a/services/web/test/unit/coffee/Subscription/SubscriptionControllerTests.coffee +++ b/services/web/test/unit/coffee/Subscription/SubscriptionControllerTests.coffee @@ -99,7 +99,11 @@ describe "SubscriptionController", -> @UserGetter.getUser.callCount.should.equal 1 done() - it 'should decide not to AB test the plans when signed up before 2016-10-27', (done) -> + it 'should decide not to AB test the plans when signed up before 2018-06-06', (done) -> + # Users before we introduce the test may have already seen the old variant, + # and so may react positively to a change rather than the variant itself. + # So it's more likely to skew in favour of the change + # just because change makes things 'fresh' @res.renderedVariables.shouldABTestPlans.should.equal false done()