diff --git a/services/web/public/coffee/main/plans.coffee b/services/web/public/coffee/main/plans.coffee
index 6db59aa0e4..4817c2d12c 100644
--- a/services/web/public/coffee/main/plans.coffee
+++ b/services/web/public/coffee/main/plans.coffee
@@ -333,13 +333,11 @@ define [
 			$scope.currencyCode = newCurrency
 
 		$scope.signUpNowClicked = (plan, annual)->
-			if $scope.ui.view == "annual"
-				plan = "#{plan}_annual"
-
-			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', () ->
+			if $scope.ui.view == "annual"
+				plan = "#{plan}_annual"
+			event_tracking.send   'subscription-funnel', 'sign_up_now_button', plan
 
 		$scope.switchToMonthly = ->
 			$scope.ui.view = "monthly"