Parcourir la source

Merge pull request #24609 from overleaf/rh-bs5-b2c-teardown

Teardown Bootstrap 5 B2C feature flag

GitOrigin-RevId: 4021820bc3b207dae9fffd23dfac778746fa8f20
roo hutton il y a 1 an
Parent
commit
774292d8ba

+ 0 - 8
services/web/app/src/Features/TokenAccess/TokenAccessController.mjs

@@ -20,7 +20,6 @@ import { getSafeAdminDomainRedirect } from '../Helpers/UrlHelper.js'
 import UserGetter from '../User/UserGetter.js'
 import UserGetter from '../User/UserGetter.js'
 import Settings from '@overleaf/settings'
 import Settings from '@overleaf/settings'
 import LimitationsManager from '../Subscription/LimitationsManager.js'
 import LimitationsManager from '../Subscription/LimitationsManager.js'
-import SplitTestHandler from '../SplitTests/SplitTestHandler.js'
 
 
 const orderedPrivilegeLevels = [
 const orderedPrivilegeLevels = [
   PrivilegeLevels.NONE,
   PrivilegeLevels.NONE,
@@ -112,13 +111,6 @@ async function tokenAccessPage(req, res, next) {
       }
       }
     }
     }
 
 
-    // Populates splitTestVariants with a value for the split test name and allows
-    // Pug to read it
-    await SplitTestHandler.promises.getAssignment(
-      req,
-      res,
-      'bs5-misc-pages-b2c'
-    )
     res.render('project/token/access-react', {
     res.render('project/token/access-react', {
       postUrl: makePostUrl(token),
       postUrl: makePostUrl(token),
     })
     })

+ 0 - 6
services/web/app/src/Features/User/UserEmailsController.js

@@ -480,10 +480,6 @@ async function confirmSecondaryEmailPage(req, res) {
     return res.redirect(redirectURL)
     return res.redirect(redirectURL)
   }
   }
 
 
-  // Populates splitTestVariants with a value for the split test name and allows
-  // Pug to read it
-  await SplitTestHandler.promises.getAssignment(req, res, 'bs5-misc-pages-b2c')
-
   AnalyticsManager.recordEventForUserInBackground(
   AnalyticsManager.recordEventForUserInBackground(
     userId,
     userId,
     'confirm-secondary-email-page-displayed'
     'confirm-secondary-email-page-displayed'
@@ -511,8 +507,6 @@ async function addSecondaryEmailPage(req, res) {
     'add-secondary-email-page-displayed'
     'add-secondary-email-page-displayed'
   )
   )
 
 
-  await SplitTestHandler.promises.getAssignment(req, res, 'bs5-misc-pages-b2c')
-
   res.render('user/addSecondaryEmail')
   res.render('user/addSecondaryEmail')
 }
 }
 
 

+ 0 - 1
services/web/app/views/project/token/access-react.pug

@@ -8,7 +8,6 @@ block vars
 	- var suppressCookieBanner = true
 	- var suppressCookieBanner = true
 	- var suppressSkipToContent = true
 	- var suppressSkipToContent = true
 	- bootstrap5PageStatus = 'enabled'
 	- bootstrap5PageStatus = 'enabled'
-	- bootstrap5PageSplitTest = 'bs5-misc-pages-b2c'
 		
 		
 block append meta
 block append meta
 	meta(name="ol-postUrl" data-type="string" content=postUrl)
 	meta(name="ol-postUrl" data-type="string" content=postUrl)

+ 0 - 1
services/web/app/views/user/addSecondaryEmail.pug

@@ -4,7 +4,6 @@ block vars
 	- var suppressNavbar = true
 	- var suppressNavbar = true
 	- var suppressSkipToContent = true
 	- var suppressSkipToContent = true
 	- bootstrap5PageStatus = 'enabled'
 	- bootstrap5PageStatus = 'enabled'
-	- bootstrap5PageSplitTest = 'bs5-misc-pages-b2c'
 
 
 block entrypointVar
 block entrypointVar
 	- entrypoint = 'pages/user/add-secondary-email'
 	- entrypoint = 'pages/user/add-secondary-email'

+ 0 - 1
services/web/app/views/user/confirmSecondaryEmail.pug

@@ -4,7 +4,6 @@ block vars
 	- var suppressNavbar = true
 	- var suppressNavbar = true
 	- var suppressSkipToContent = true
 	- var suppressSkipToContent = true
 	- bootstrap5PageStatus = 'enabled'
 	- bootstrap5PageStatus = 'enabled'
-	- bootstrap5PageSplitTest = 'bs5-misc-pages-b2c'
 
 
 block entrypointVar
 block entrypointVar
 	- entrypoint = 'pages/user/confirm-secondary-email'
 	- entrypoint = 'pages/user/confirm-secondary-email'