From 97da5868778ee7bcd66cd54cb9ba26a2af32da18 Mon Sep 17 00:00:00 2001
From: Jakob Ackermann <jakob.ackermann@overleaf.com>
Date: Mon, 19 Jul 2021 11:27:42 +0200
Subject: [PATCH] Merge pull request #4326 from
 overleaf/pr-avoid-double-stringify-compile-events

Remove controller-level `stringify` for compile events

GitOrigin-RevId: 1b33b5204a846dc4f25dcfbad02c1edc1e010e9f
---
 .../web/frontend/js/ide/pdf/controllers/PdfController.js    | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/services/web/frontend/js/ide/pdf/controllers/PdfController.js b/services/web/frontend/js/ide/pdf/controllers/PdfController.js
index 12fe5ab8fd..d7c23faecc 100644
--- a/services/web/frontend/js/ide/pdf/controllers/PdfController.js
+++ b/services/web/frontend/js/ide/pdf/controllers/PdfController.js
@@ -732,11 +732,7 @@ App.controller(
           newLogsUI: window.showNewLogsUI,
           subvariant: window.showNewLogsUI ? window.logsUISubvariant : null,
         }
-        eventTracking.sendMBSampled(
-          'compile-result',
-          JSON.stringify(metadata),
-          0.01
-        )
+        eventTracking.sendMBSampled('compile-result', metadata, 0.01)
       }
     }