feat: use Segment (batch 1)

app/components/Nav/App/index.js

@@ -100,7 +100,7 @@ import AsyncStorage from '../../../store/async-storage-wrapper';
 import ShowIpfsGatewaySheet from '../../Views/ShowIpfsGatewaySheet/ShowIpfsGatewaySheet';
 import ShowDisplayNftMediaSheet from '../../Views/ShowDisplayMediaNFTSheet/ShowDisplayNFTMediaSheet';
 import AmbiguousAddressSheet from '../../../../app/components/Views/Settings/Contacts/AmbiguousAddressSheet/AmbiguousAddressSheet';
-import { useMetrics } from '../../hooks/useMetrics';
+import { MetaMetrics } from '../../../core/Analytics';
 
 const clearStackNavigatorOptions = {
   headerShown: false,
@@ -230,7 +230,6 @@ const VaultRecoveryFlow = () => (
 );
 
 const App = ({ userLoggedIn }) => {
-  const { getMetaMetricsId } = useMetrics();
   const animationRef = useRef(null);
   const animationNameRef = useRef(null);
   const opacity = useRef(new Animated.Value(1)).current;
@@ -370,15 +369,14 @@ const App = ({ userLoggedIn }) => {
 
   useEffect(() => {
     const initAnalytics = async () => {
-      const metaMetricsId = await getMetaMetricsId();
-      Logger.log(`MetaMetrics initialised with ID: ${metaMetricsId}`);
+      await MetaMetrics.getInstance().configure();
       await Analytics.init();
     };
 
     initAnalytics().catch((err) => {
       Logger.error(err, 'Error initializing analytics');
     });
-  }, [getMetaMetricsId]);
+  }, []);
 
   useEffect(() => {
     // Init SDKConnect only if the navigator is ready, user is onboarded, and SDK is not initialized.

app/components/Views/Settings/SecuritySettings/SecuritySettings.tsx

@@ -231,7 +231,7 @@ const Settings: React.FC = () => {
     setHintText(manualBackup);
   }, []);
 
-  const checkAnalyticsEnabled = useCallback(async () => {
+  const checkAnalyticsEnabled = useCallback(() => {
     trackEvent(MetaMetricsEvents.VIEW_SECURITY_SETTINGS, {});
     setAnalyticsEnabled(isEnabled());
   }, [trackEvent, isEnabled]);

app/core/Analytics/MetaMetrics.ts

@@ -276,8 +276,6 @@ class MetaMetrics implements IMetaMetrics {
     } else {
       this.metametricsId = metametricsId;
     }
-
-    if (__DEV__) Logger.log(`Current MetaMatrics ID: ${this.metametricsId}`);
     return this.metametricsId;
   };
 
@@ -507,6 +505,8 @@ class MetaMetrics implements IMetaMetrics {
         await this.#getDeleteRegulationDateFromPrefs();
       this.dataRecorded = await this.#getIsDataRecordedFromPrefs();
       this.#isConfigured = true;
+      if (__DEV__)
+        Logger.log(`MetaMetrics configured with ID: ${this.metametricsId}`);
     } catch (error: any) {
       Logger.error(error, 'Error initializing MetaMetrics');
     }

app/util/metrics/TrackAfterInteraction/trackAfterInteractions.test.ts

@@ -1,4 +1,5 @@
 import trackAfterInteractions from './trackAfterInteractions';
+import { IMetaMetricsEvent } from '../../../core/Analytics';
 
 const { InteractionManager } = jest.requireActual('react-native');
 InteractionManager.runAfterInteractions = jest.fn(async (callback) =>
@@ -25,7 +26,7 @@ describe('trackAfterInteractions', () => {
   });
 
   it('calls saveOnboardingEvent when metrics is not enabled', async () => {
-    const mockEvent = { category: 'testEvent' };
+    const mockEvent: IMetaMetricsEvent = { category: 'testEvent' };
     const mockProperties = { prop: 'testProp' };
 
     mockEnabled.mockReturnValue(false);
@@ -36,37 +37,31 @@ describe('trackAfterInteractions', () => {
       mockSaveOnboardingEvent,
     );
 
-    expect(mockSaveOnboardingEvent).toHaveBeenCalledWith(mockEvent.category);
+    expect(mockSaveOnboardingEvent).toHaveBeenCalledWith(mockEvent);
     expect(mockTrackEvent).not.toHaveBeenCalled();
   });
 
   it('call trackEvent when metrics is not enabled but saveOnboardingEvent is not defined', async () => {
-    const mockEvent = { category: 'testEvent' };
+    const mockEvent: IMetaMetricsEvent = { category: 'testEvent' };
     const mockProperties = { prop: 'testProp' };
 
     mockEnabled.mockReturnValue(true);
 
     await trackAfterInteractions(mockEvent, mockProperties);
 
     expect(mockSaveOnboardingEvent).not.toHaveBeenCalledWith();
-    expect(mockTrackEvent).toHaveBeenCalledWith(
-      mockEvent.category,
-      mockProperties,
-    );
+    expect(mockTrackEvent).toHaveBeenCalledWith(mockEvent, mockProperties);
   });
 
   it('call trackEvent when metrics is enabled', async () => {
-    const mockEvent = { category: 'testEvent' };
+    const mockEvent: IMetaMetricsEvent = { category: 'testEvent' };
     const mockProperties = { prop: 'testProp' };
 
     mockEnabled.mockReturnValue(true);
 
     await trackAfterInteractions(mockEvent, mockProperties);
 
     expect(mockSaveOnboardingEvent).not.toHaveBeenCalledWith();
-    expect(mockTrackEvent).toHaveBeenCalledWith(
-      mockEvent.category,
-      mockProperties,
-    );
+    expect(mockTrackEvent).toHaveBeenCalledWith(mockEvent, mockProperties);
   });
 });

app/util/metrics/TrackAfterInteraction/trackAfterInteractions.ts

@@ -11,8 +11,8 @@ import { JsonMap } from '@segment/analytics-react-native';
 const trackAfterInteractions = async (
   event: IMetaMetricsEvent,
   properties: JsonMap = {},
-  saveOnboardingEvent?: (event: string) => {
-    event: object;
+  saveOnboardingEvent?: (event: IMetaMetricsEvent) => {
+    event: IMetaMetricsEvent;
     type: string;
   },
 ): Promise<void> =>
@@ -21,7 +21,7 @@ const trackAfterInteractions = async (
     const isOnboardingDelayedEvent =
       !metrics.isEnabled() && saveOnboardingEvent;
     if (isOnboardingDelayedEvent) {
-      saveOnboardingEvent(event.category);
+      saveOnboardingEvent(event);
     } else {
       metrics.trackEvent(event, properties);
     }