Fixed metrics

MetaMask · Feb 14, 2023 · 46f8e6b · 46f8e6b
1 parent 5f6c703
commit 46f8e6b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/app/scripts/controllers/metametrics.js b/app/scripts/controllers/metametrics.js
@@ -125,8 +125,6 @@ export default class MetaMetricsController {
       environment === 'production' ? version : `${version}-${environment}`;
     this.extension = extension;
     this.environment = environment;
-    this.transactionSecurityCheckEnabled =
-      prefState.transactionSecurityCheckEnabled;
 
     const abandonedFragments = omitBy(initState?.fragments, 'persist');
     const segmentApiCalls = initState?.segmentApiCalls || {};
@@ -145,12 +143,9 @@ export default class MetaMetricsController {
       },
     });
 
-    preferencesStore.subscribe(
-      ({ currentLocale, transactionSecurityCheckEnabled }) => {
-        this.locale = currentLocale.replace('_', '-');
-        this.transactionSecurityCheckEnabled = transactionSecurityCheckEnabled;
-      },
-    );
+    preferencesStore.subscribe(({ currentLocale }) => {
+      this.locale = currentLocale.replace('_', '-');
+    });
 
     onNetworkDidChange(() => {
       this.chainId = getCurrentChainId();
@@ -721,7 +716,7 @@ export default class MetaMetricsController {
       [TRAITS.THREE_BOX_ENABLED]: false, // deprecated, hard-coded as false
       [TRAITS.THEME]: metamaskState.theme || 'default',
       [TRAITS.TOKEN_DETECTION_ENABLED]: metamaskState.useTokenDetection,
-      [TRAITS.SECURITY_PROVIDER]: this.transactionSecurityCheckEnabled
+      [TRAITS.SECURITY_PROVIDER]: metamaskState.transactionSecurityCheckEnabled
         ? ['opensea']
         : [],
     };

diff --git a/shared/constants/metametrics.js b/shared/constants/metametrics.js
@@ -240,6 +240,7 @@ export const TRAITS = {
  *  enabled? (deprecated)
  * @property {string} [theme] - which theme the user has selected
  * @property {boolean} [token_detection_enabled] - does the user have token detection is enabled?
+ * @property {Array<string>} [security_provider] - whether security provider feature toggle is on or off
  */
 
 // Mixpanel converts the zero address value to a truly anonymous event, which