From b48529497def4a18c8ab8b8544a86265fb992082 Mon Sep 17 00:00:00 2001 From: Alexandre Bourdin Date: Tue, 31 May 2022 10:11:08 +0200 Subject: [PATCH] Merge pull request #8232 from overleaf/ab-fix-anonymous-users-assignment Fix the condition to assign anonymous users to default for alpha/beta phases GitOrigin-RevId: 4ffaf2cceeadbfc76120c874ee2c86f747034e9f --- services/web/app/src/Features/SplitTests/SplitTestHandler.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/services/web/app/src/Features/SplitTests/SplitTestHandler.js b/services/web/app/src/Features/SplitTests/SplitTestHandler.js index d35c030147..d5a362b185 100644 --- a/services/web/app/src/Features/SplitTests/SplitTestHandler.js +++ b/services/web/app/src/Features/SplitTests/SplitTestHandler.js @@ -207,9 +207,8 @@ async function _getAssignmentMetadata(analyticsId, user, splitTest) { const currentVersion = splitTest.getCurrentVersion() const phase = currentVersion.phase if ( - user && - ((phase === ALPHA_PHASE && !user.alphaProgram) || - (phase === BETA_PHASE && !user.betaProgram)) + (phase === ALPHA_PHASE && !user?.alphaProgram) || + (phase === BETA_PHASE && !user?.betaProgram) ) { return { activeForUser: false,