Merge pull request #12187 from overleaf/jpa-mf-old-plans-page-annual-variant

Differentiate user that visits from before the fix and after the fix

GitOrigin-RevId: 4c3c4b47dc3329b97da79adcb2a45bf95b0ff27a

services/web/app/src/Features/Subscription/SubscriptionController.js

@@ -64,6 +64,12 @@ async function plansPage(req, res) {
       res,
       'plans-page-layout-v3'
     )
+
+    if (plansPageLayoutV3Assignment.variant === 'old-plans-page-annual') {
+      plansPageLayoutV3Assignment.variant = 'old-plans-page-annual-fixed'
+      res.locals.splitTestVariants['plans-page-layout-v3'] =
+        'old-plans-page-annual-fixed'
+    }
   } catch (error) {
     logger.error(
       { err: error },
@@ -73,7 +79,7 @@ async function plansPage(req, res) {
 
   let currentView = 'monthly'
   if (
-    plansPageLayoutV3Assignment.variant === 'old-plans-page-annual' ||
+    plansPageLayoutV3Assignment.variant === 'old-plans-page-annual-fixed' ||
     plansPageLayoutV3Assignment.variant === 'new-plans-page'
   ) {
     currentView = 'annual'

services/web/app/views/subscriptions/plans-marketing.pug

@@ -63,7 +63,7 @@ block content
 								h2 #{translate('compare_plan_features')}
 					.row
 						.col-md-6.col-md-offset-3
-							if (plansPageLayoutV3Variant === 'old-plans-page-annual')
+							if (plansPageLayoutV3Variant === 'old-plans-page-annual-fixed')
 								+plan_switch_annual_default('table')
 							else
 								+plan_switch('table')

services/web/app/views/subscriptions/plans-marketing/_mixins.pug

@@ -240,7 +240,7 @@ mixin allCardsAndControls(controlsRowSpaced, listLocation)
 	- var location = listLocation ? 'card_' + listLocation : 'card'
 	.row.top-switch(class=(controlsRowSpaced ? "row-spaced" : ""))
 		.col-md-6.col-md-offset-3
-			if (plansPageLayoutV3Variant === 'old-plans-page-annual')
+			if (plansPageLayoutV3Variant === 'old-plans-page-annual-fixed')
 				+plan_switch_annual_default(location)
 			else
 				+plan_switch(location)