From 310cbe10bad8d648799b65c938b9a62ab0d74a2c Mon Sep 17 00:00:00 2001
From: Jessica Lawshe <5312836+lawshe@users.noreply.github.com>
Date: Wed, 13 Mar 2024 08:42:40 -0500
Subject: [PATCH] Merge pull request #17434 from
 overleaf/ab-allow-analytics-inactive-test

[web] Allow analytics to be enabled for inactive tests

GitOrigin-RevId: f854fd6532357c4291f2785b950eac2478cfbae0
---
 services/web/app/src/Features/SplitTests/SplitTestManager.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/services/web/app/src/Features/SplitTests/SplitTestManager.js b/services/web/app/src/Features/SplitTests/SplitTestManager.js
index d54423c46a..1fa74d4937 100644
--- a/services/web/app/src/Features/SplitTests/SplitTestManager.js
+++ b/services/web/app/src/Features/SplitTests/SplitTestManager.js
@@ -152,7 +152,7 @@ async function updateSplitTestConfig({ name, configuration, comment }, userId) {
     versionNumber: lastVersion.versionNumber + 1,
     phase: configuration.phase,
     active: configuration.active,
-    analyticsEnabled: configuration.active && configuration.analyticsEnabled,
+    analyticsEnabled: configuration.analyticsEnabled,
     variants: updatedVariants,
     author: userId,
     comment,