From 8097e6ad712bb7bc0c5890ca23cabe7b891d1154 Mon Sep 17 00:00:00 2001
From: Henry Oswald <henry.oswald@sharelatex.com>
Date: Tue, 21 Oct 2014 13:48:46 +0100
Subject: [PATCH] made 7 day free trial the default

---
 .../coffee/Features/Compile/CompileController.coffee |  2 +-
 services/web/app/views/subscriptions/new.jade        |  5 -----
 .../views/subscriptions/successful_subscription.jade |  2 --
 services/web/public/coffee/ide.coffee                |  9 +--------
 services/web/public/coffee/main/plans.coffee         | 12 +++---------
 5 files changed, 5 insertions(+), 25 deletions(-)

diff --git a/services/web/app/coffee/Features/Compile/CompileController.coffee b/services/web/app/coffee/Features/Compile/CompileController.coffee
index d3b470b27f..670cb4bec9 100755
--- a/services/web/app/coffee/Features/Compile/CompileController.coffee
+++ b/services/web/app/coffee/Features/Compile/CompileController.coffee
@@ -64,12 +64,12 @@ module.exports = CompileController =
 		CompileController.proxyToClsi(req.url, req, res, next)
 
 	proxyToClsi: (url, req, res, next = (error) ->) ->
-		logger.log url: url, "proxying to CLSI"
 		if req.session.compileGroup == "priority"
 			compilerUrl = Settings.apis.clsi_priority.url
 		else
 			compilerUrl = Settings.apis.clsi.url
 		url = "#{compilerUrl}#{url}"
+		logger.log url: url, "proxying to CLSI"
 		oneMinute = 60 * 1000
 		proxy = request(url: url, method: req.method, timeout: oneMinute)
 		proxy.pipe(res)
diff --git a/services/web/app/views/subscriptions/new.jade b/services/web/app/views/subscriptions/new.jade
index 6a3c845095..d954f32f2b 100644
--- a/services/web/app/views/subscriptions/new.jade
+++ b/services/web/app/views/subscriptions/new.jade
@@ -80,12 +80,7 @@ block content
 		Recurly.buildSubscriptionForm(recurlySubscriptionFormConfig);
 
 		window.ab = [
-			{step:1, bucket:"14d", testName:"trial_len"},
-			{step:1, bucket:"7d", testName:"trial_len"},
-
 			{step:1, bucket:"eu-eu", testName:"multi_currency"},
 			{step:1, bucket:"eu-usd", testName:"multi_currency"}
-
-
 		]
 
diff --git a/services/web/app/views/subscriptions/successful_subscription.jade b/services/web/app/views/subscriptions/successful_subscription.jade
index 5d989fdf6e..a9c4699b10 100644
--- a/services/web/app/views/subscriptions/successful_subscription.jade
+++ b/services/web/app/views/subscriptions/successful_subscription.jade
@@ -35,8 +35,6 @@ block content
 
 	script(type="text/javascript").
 		window.ab = [
-			{step:2, bucket:"14d", testName:"trial_len"},
-			{step:2, bucket:"7d", testName:"trial_len"},
 			{step:2, bucket:"eu-eu", testName:"multi_currency"},
 			{step:2, bucket:"eu-usd", testName:"multi_currency"},
 
diff --git a/services/web/public/coffee/ide.coffee b/services/web/public/coffee/ide.coffee
index 9446d7f87d..84121d164f 100644
--- a/services/web/public/coffee/ide.coffee
+++ b/services/web/public/coffee/ide.coffee
@@ -66,14 +66,7 @@ define [
 
 		$scope.startFreeTrial = (source) ->
 
-			buckets = [
-				{ bucketName:"7d", planCode: "student_free_trial_7_days" }
-				{ bucketName:"14d", planCode: "student_free_trial_14_days" }
-			]
-			bucket = abTestManager.getABTestBucket "trial_len", buckets
-			abTestManager.processTestWithStep("trial_len", bucket.bucketName, 0)
-
-			window.open("/user/subscription/new?planCode=#{bucket.planCode}")
+			window.open("/user/subscription/new?planCode=student_free_trial_7_days")
 			$scope.startedFreeTrial = true
 
 		window._ide = ide
diff --git a/services/web/public/coffee/main/plans.coffee b/services/web/public/coffee/main/plans.coffee
index e8a014a43e..d94869e4d3 100644
--- a/services/web/public/coffee/main/plans.coffee
+++ b/services/web/public/coffee/main/plans.coffee
@@ -58,10 +58,6 @@ define [
 		$scope.currencyCode = MultiCurrencyPricing.currencyCode
 
 
-		buckets = [
-			{ bucketName:"7d", queryString: "_free_trial_7_days", trial_len:7 }
-			{ bucketName:"14d", queryString: "_free_trial_14_days", trial_len:14 }
-		]
 
 		if MultiCurrencyPricing.currencyCode != "USD"
 			currencyBuckets = [
@@ -72,10 +68,9 @@ define [
 			$scope.currencyCode = multiCurrencyBucket.currency
 
 
-		bucket = abTestManager.getABTestBucket "trial_len", buckets
 
-		$scope.trial_len = bucket.trial_len
-		$scope.planQueryString = bucket.queryString
+		$scope.trial_len = 7
+		$scope.planQueryString = '_free_trial_7_days'
 
 		$scope.ui =
 			view: "monthly"
@@ -91,8 +86,7 @@ define [
 
 			if $scope.ui.view == "annual"
 				plan = "#{plan}_annual"
-			else
-				abTestManager.processTestWithStep("trial_len", bucket.bucketName, 0)
+			
 			event_tracking.send 'subscription-funnel', 'sign_up_now_button', plan
 
 		$scope.switchToMonthly = ->