Просмотр исходного кода

Merge pull request #14866 from overleaf/jdt-hist-rename-test

fix: rename the history view split test so we can reroll it
GitOrigin-RevId: fd42fab5b6fd715b89688b401596962453b1d002
Jimmy Domagala-Tang 2 лет назад
Родитель
Сommit
ab4f0d6c5f

+ 1 - 1
services/web/app/src/Features/Project/ProjectController.js

@@ -702,7 +702,7 @@ const ProjectController = {
           SplitTestHandler.getAssignment(
             req,
             res,
-            'history-view',
+            'history-view-version',
             (error, assignment) => {
               // do not fail editor load if assignment fails
               if (error) {

+ 1 - 1
services/web/frontend/js/features/history/components/diff-view/toolbar/toolbar.tsx

@@ -19,7 +19,7 @@ export default function Toolbar({ diff, selection }: ToolbarProps) {
   return (
     <div className="history-react-toolbar">
       <SplitTestBadge
-        splitTestName="history-view"
+        splitTestName="history-view-version"
         displayOnVariants={['react']}
       />
       <ToolbarDatetime selection={selection} />

+ 3 - 1
services/web/frontend/js/ide/history/HistoryV2Manager.js

@@ -126,7 +126,9 @@ export default HistoryManager = (function () {
     }
 
     isReact() {
-      return getMeta('ol-splitTestVariants')?.['history-view'] === 'react'
+      return (
+        getMeta('ol-splitTestVariants')?.['history-view-version'] === 'react'
+      )
     }
 
     hardReset() {