Browse Source

Move convert to top

Shane Kilkelly 10 năm trước cách đây
mục cha
commit
c1ebc65a3b

+ 2 - 4
services/web/public/coffee/main/plans.coffee

@@ -333,13 +333,11 @@ define [
 			$scope.currencyCode = newCurrency
 			$scope.currencyCode = newCurrency
 
 
 		$scope.signUpNowClicked = (plan, annual)->
 		$scope.signUpNowClicked = (plan, annual)->
+			if $scope.shouldABTestPlans and plan in ['student', 'collaborator']
+				sixpack.convert 'plans-1610', () ->
 			if $scope.ui.view == "annual"
 			if $scope.ui.view == "annual"
 				plan = "#{plan}_annual"
 				plan = "#{plan}_annual"
-
 			event_tracking.send   'subscription-funnel', 'sign_up_now_button', plan
 			event_tracking.send   'subscription-funnel', 'sign_up_now_button', plan
-			# TODO: double check this is correct
-			if $scope.shouldABTestPlans and plan in ['student', 'collaborator']
-				sixpack.convert 'plans-1610', () ->
 
 
 		$scope.switchToMonthly = ->
 		$scope.switchToMonthly = ->
 			$scope.ui.view = "monthly"
 			$scope.ui.view = "monthly"