From ede801e1439037a6636665fabeb0859b455d5b79 Mon Sep 17 00:00:00 2001 From: Ariella Vu <20778143+digiwand@users.noreply.github.com> Date: Tue, 14 Jun 2022 11:00:11 -0500 Subject: [PATCH] MetaMetrics: fix onCancel trackEvent (#14935) should mirror first-time-flow/metametrics-opt-in logic --- ui/pages/onboarding-flow/metametrics/metametrics.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/ui/pages/onboarding-flow/metametrics/metametrics.js b/ui/pages/onboarding-flow/metametrics/metametrics.js index b27c12b4d..e9928577c 100644 --- a/ui/pages/onboarding-flow/metametrics/metametrics.js +++ b/ui/pages/onboarding-flow/metametrics/metametrics.js @@ -43,8 +43,10 @@ export default function OnboardingMetametrics() { const onConfirm = async () => { const [, metaMetricsId] = await dispatch(setParticipateInMetaMetrics(true)); + const isInitiallyNotParticipating = !participateInMetaMetrics; + try { - if (!participateInMetaMetrics) { + if (isInitiallyNotParticipating) { trackEvent( { category: EVENT.CATEGORIES.ONBOARDING, @@ -83,8 +85,11 @@ export default function OnboardingMetametrics() { const onCancel = async () => { await dispatch(setParticipateInMetaMetrics(false)); + const isInitiallyParticipatingOrNotSet = + participateInMetaMetrics === null || participateInMetaMetrics; + try { - if (!participateInMetaMetrics) { + if (isInitiallyParticipatingOrNotSet) { trackEvent( { category: EVENT.CATEGORIES.ONBOARDING,