Merge pull request #9152 from LedgerHQ/fix/llm-analytics-console-deep…

…-props fix(llm): 🩹 fix the analytics console for deep properties
LedgerHQ · Feb 7, 2025 · 2b90b42 · 2b90b42
2 parents 9bfe259 + 9bd9938
commit 2b90b42
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.changeset/blue-humans-destroy.md b/.changeset/blue-humans-destroy.md
@@ -0,0 +1,5 @@
+---
+"live-mobile": minor
+---
+
+Fix the analytics console for deep properties
diff --git a/apps/ledger-live-mobile/src/components/AnalyticsConsole/Event.tsx b/apps/ledger-live-mobile/src/components/AnalyticsConsole/Event.tsx
@@ -22,10 +22,7 @@ const Event: React.FC<Props> = ({
   const propertiesText = useMemo(
     () =>
       propertiesToDisplay
-        ? JSON.stringify(propertiesToDisplay, Object.keys(propertiesToDisplay).sort(), 2)
-            .split("\n")
-            .slice(1, -1)
-            .join("\n")
+        ? JSON.stringify(propertiesToDisplay, null, 2).replace(/^{\n|\n}$/g, "")
         : null,
     [propertiesToDisplay],
   );