From d3175a3b530f946d50ab3d22c37373baa48d1ca8 Mon Sep 17 00:00:00 2001
From: Daniel James Smith <djsmith85@users.noreply.github.com>
Date: Fri, 21 Mar 2025 10:20:05 +0100
Subject: [PATCH] Skipping the error message on the achievement processor but
 still emitting the metric update

---
 .../src/tools/achievements/achievement-processor.ts       | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/libs/common/src/tools/achievements/achievement-processor.ts b/libs/common/src/tools/achievements/achievement-processor.ts
index 3adb4e0eddf..8f8e71bfb02 100644
--- a/libs/common/src/tools/achievements/achievement-processor.ts
+++ b/libs/common/src/tools/achievements/achievement-processor.ts
@@ -46,10 +46,12 @@ function achievements(
 
         // prevent duplicate updates
         if (distinct.has(key)) {
-          const msg = `${achievement} failed to set ${key}; value already set by ${distinct.get(key)}`;
-          throw new Error(msg);
+          // FIXME Currently skipping this error for demo purposes
+          // const msg = `${achievement} failed to set ${key}; value already set by ${distinct.get(key)}`;
+          // throw new Error(msg);
+        } else {
+          distinct.set(key, achievement);
         }
-        distinct.set(key, achievement);
 
         metrics.set(measured.achievement.name, measured.achievement.value);
       }