fix: metrics event not fired when blockaid is disabled (#8088)

MetaMask · Dec 18, 2023 · c4b18ed · c4b18ed
1 parent 1b4339c
commit c4b18ed
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/app/components/Views/Settings/ExperimentalSettings/BlockaidIndicator/BlockaidIndicator.tsx b/app/components/Views/Settings/ExperimentalSettings/BlockaidIndicator/BlockaidIndicator.tsx
@@ -69,7 +69,7 @@ const BlockaidIndicator = ({ navigation }: Props) => {
       AnalyticsV2.trackEvent(
         MetaMetricsEvents.SETTINGS_EXPERIMENTAL_SECURITY_ALERTS_ENABLED,
         {
-          security_alerts_enabled: !securityAlertsEnabled,
+          security_alerts_enabled: true,
         },
       );
     });

diff --git a/app/components/Views/Settings/ExperimentalSettings/index.tsx b/app/components/Views/Settings/ExperimentalSettings/index.tsx
@@ -8,6 +8,8 @@ import Text, {
   TextVariant,
   TextColor,
 } from '../../../../component-library/components/Texts/Text';
+import AnalyticsV2 from '../../../../util/analyticsV2';
+import { MetaMetricsEvents } from '../../../../core/Analytics';
 import { getNavigationOptionsTitle } from '../../../UI/Navbar';
 import StyledButton from '../../../UI/StyledButton';
 import SECURITY_ALERTS_TOGGLE_TEST_ID from './constants';
@@ -39,6 +41,12 @@ const ExperimentalSettings = ({ navigation, route }: Props) => {
         ppomInitializationCompleted: false,
       });
       PreferencesController?.setSecurityAlertsEnabled(!securityAlertsEnabled);
+      AnalyticsV2.trackEvent(
+        MetaMetricsEvents.SETTINGS_EXPERIMENTAL_SECURITY_ALERTS_ENABLED,
+        {
+          security_alerts_enabled: false,
+        },
+      );
     } else {
       navigation.navigate(Routes.MODAL.ROOT_MODAL_FLOW, {
         screen: Routes.SHEET.BLOCKAID_INDICATOR,