Skip to content

Commit

Permalink
Ensure user traits set before opting in to participate in metametrics…
Browse files Browse the repository at this point in the history
… get properly updated upon participate in metametrics
  • Loading branch information
danjm committed Jan 27, 2025
1 parent 64e2c3f commit a7887e3
Showing 1 changed file with 8 additions and 4 deletions.
12 changes: 8 additions & 4 deletions app/scripts/controllers/metametrics-controller.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1239,7 +1239,7 @@ export default class MetaMetricsController extends BaseController<
),
};

if (!previousUserTraits) {
if (!previousUserTraits && metamaskState.participateInMetaMetrics) {
this.update((state) => {
state.previousUserTraits = currentTraits;
});
Expand All @@ -1252,9 +1252,13 @@ export default class MetaMetricsController extends BaseController<
const previous = previousUserTraits[k];
return !isEqual(previous, v);
});
this.update((state) => {
state.previousUserTraits = currentTraits;
});

if (metamaskState.participateInMetaMetrics) {
this.update((state) => {
state.previousUserTraits = currentTraits;
});
}

return updates;
}

Expand Down

0 comments on commit a7887e3

Please sign in to comment.