From b58fd6f9d726910e109281bc6c2bc7e1e6ce9d7e Mon Sep 17 00:00:00 2001
From: Krzysztof Modras <1228153+chrmod@users.noreply.github.com>
Date: Thu, 13 Feb 2025 14:35:13 +0100
Subject: [PATCH] Fixes for 10.4.25 (#2253)

* Fix: remote config should not require feedback consent

* Fix: engaged telemetry should be sent asap
---
 src/background/config.js            | 4 ++--
 src/background/telemetry/metrics.js | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/background/config.js b/src/background/config.js
index a1f1fe1cb..73b4d7661 100644
--- a/src/background/config.js
+++ b/src/background/config.js
@@ -105,6 +105,6 @@ export default async function syncConfig() {
   }
 }
 
-OptionsObserver.addListener(function config({ terms, feedback }) {
-  if (terms && feedback) syncConfig();
+OptionsObserver.addListener(function config({ terms }) {
+  if (terms) syncConfig();
 });
diff --git a/src/background/telemetry/metrics.js b/src/background/telemetry/metrics.js
index f3be36295..abe61771c 100644
--- a/src/background/telemetry/metrics.js
+++ b/src/background/telemetry/metrics.js
@@ -311,7 +311,7 @@ export default class Metrics {
 
     // Protect against calling events immediately after install for all frequencies
     // They should trigger on the trailing edge of the frequency
-    if (!this.storage[key]) {
+    if (!this.storage[key] && type !== 'engaged') {
       this.storage[key] = now;
       this.saveStorage(this.storage);
     }