From c4e0294d3600d303979170c86310ae122c3f3393 Mon Sep 17 00:00:00 2001
From: M Fahru <m.fahru@overleaf.com>
Date: Wed, 4 Dec 2024 09:51:59 -0700
Subject: [PATCH] Merge pull request #22027 from
 overleaf/mf-activate-plans-page-new-design-variant

[web] Activate plans page new-design variant for all users

GitOrigin-RevId: 7da3c973cded8597cadd691f5d6e928d31e8d26f
---
 .../Subscription/SubscriptionController.js    |  7 -------
 .../Subscription/SubscriptionRouter.mjs       | 19 -------------------
 .../SubscriptionControllerTests.js            |  1 -
 3 files changed, 27 deletions(-)

diff --git a/services/web/app/src/Features/Subscription/SubscriptionController.js b/services/web/app/src/Features/Subscription/SubscriptionController.js
index 5d4232623b..e9ece23d40 100644
--- a/services/web/app/src/Features/Subscription/SubscriptionController.js
+++ b/services/web/app/src/Features/Subscription/SubscriptionController.js
@@ -163,13 +163,6 @@ async function plansPage(req, res) {
 }
 
 async function plansPageLightDesign(req, res) {
-  const splitTestActive = await SplitTestHandler.promises.isSplitTestActive(
-    'website-redesign-plans'
-  )
-
-  if (!splitTestActive && req.query.preview !== 'true') {
-    return res.redirect(302, '/user/subscription/plans')
-  }
   const { currency, countryCode, geoPricingLATAMTestVariant } =
     await _getRecommendedCurrency(req, res)
 
diff --git a/services/web/app/src/Features/Subscription/SubscriptionRouter.mjs b/services/web/app/src/Features/Subscription/SubscriptionRouter.mjs
index 3e7a585509..75d5f4dda8 100644
--- a/services/web/app/src/Features/Subscription/SubscriptionRouter.mjs
+++ b/services/web/app/src/Features/Subscription/SubscriptionRouter.mjs
@@ -24,18 +24,6 @@ export default {
       return
     }
 
-    webRouter.get(
-      '/user/subscription/plans',
-      RateLimiterMiddleware.rateLimit(subscriptionRateLimiter),
-      SubscriptionController.plansPage
-    )
-
-    webRouter.get(
-      '/user/subscription/plans-3',
-      RateLimiterMiddleware.rateLimit(subscriptionRateLimiter),
-      SubscriptionController.plansPageLightDesign
-    )
-
     webRouter.get(
       '/user/subscription',
       AuthenticationController.requireLogin(),
@@ -44,13 +32,6 @@ export default {
       SubscriptionController.userSubscriptionPage
     )
 
-    webRouter.get(
-      '/user/subscription/choose-your-plan',
-      AuthenticationController.requireLogin(),
-      RateLimiterMiddleware.rateLimit(subscriptionRateLimiter),
-      SubscriptionController.interstitialPaymentPage
-    )
-
     webRouter.get(
       '/user/subscription/thank-you',
       AuthenticationController.requireLogin(),
diff --git a/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js b/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
index 89416370a0..4450e862db 100644
--- a/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
+++ b/services/web/test/unit/src/Subscription/SubscriptionControllerTests.js
@@ -130,7 +130,6 @@ describe('SubscriptionController', function () {
     this.SplitTestV2Hander = {
       promises: {
         getAssignment: sinon.stub().resolves({ variant: 'default' }),
-        isSplitTestActive: sinon.stub().resolves(true),
       },
     }
     this.SubscriptionHelper = {