Skip to content

Commit

Permalink
Fixes for 10.4.25 (#2253)
Browse files Browse the repository at this point in the history
* Fix: remote config should not require feedback consent

* Fix: engaged telemetry should be sent asap
  • Loading branch information
chrmod authored Feb 13, 2025
1 parent 1262a17 commit b58fd6f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/background/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -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();
});
2 changes: 1 addition & 1 deletion src/background/telemetry/metrics.js
Original file line number Diff line number Diff line change
Expand Up @@ -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);
}
Expand Down

0 comments on commit b58fd6f

Please sign in to comment.