Browse Source

Merge pull request #17167 from overleaf/jpa-hide-writefull-anon

[web] hide Writefull promotion for non-logged-in users

GitOrigin-RevId: 945b0f3da4b3bbd64e71e10f40ab6f8320efb1cd
Jakob Ackermann 2 years ago
parent
commit
4118775a49
1 changed files with 12 additions and 1 deletions
  1. 12 1
      services/web/app/src/Features/Project/ProjectController.js

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

@@ -600,7 +600,18 @@ const ProjectController = {
           SplitTestHandler.getAssignment(req, res, 'ide-page', cb)
         },
         writefullIntegrationAssignment(cb) {
-          SplitTestHandler.getAssignment(req, res, 'writefull-integration', cb)
+          if (anonymous) {
+            // Disable allocation to split test for non-logged-in users.
+            // The in-editor promotion is only relevant for logged-in users.
+            cb()
+          } else {
+            SplitTestHandler.getAssignment(
+              req,
+              res,
+              'writefull-integration',
+              cb
+            )
+          }
         },
         compileLogEventsAssignment(cb) {
           SplitTestHandler.getAssignment(req, res, 'compile-log-events', cb)