update

Author: jpuri

ui/pages/confirmations/hooks/transactions/dapp-swap-comparison/useDappSwapComparisonInfo.ts

@@ -63,18 +63,20 @@ export function useDappSwapComparisonInfo() {
 
   const { commands, quotesInput, amountMin, tokenAddresses } = useMemo(() => {
     try {
-      updateRequestDetectionLatency();
       let transactionData = data;
       if (nestedTransactions?.length) {
         transactionData = nestedTransactions?.find(({ data: trxnData }) =>
           trxnData?.startsWith(FOUR_BYTE_EXECUTE_SWAP_CONTRACT),
         )?.data;
       }
-      return getDataFromSwap(
+      const result = getDataFromSwap(
         chainId,
         transactionData,
         txParams?.from as string,
       );
+      console.log('-----------inside getDataFromSwap', result);
+      updateRequestDetectionLatency();
+      return result;
     } catch (error) {
       captureException(error);
       captureDappSwapComparisonFailed('error parsing swap data');

ui/pages/confirmations/hooks/transactions/dapp-swap-comparison/useDappSwapComparisonLatencyMetrics.ts

@@ -1,4 +1,4 @@
-import { useCallback, useEffect, useRef } from 'react';
+import { useCallback, useMemo, useRef } from 'react';
 
 const N_A = 'N/A';
 
@@ -7,7 +7,8 @@ export function useDappSwapComparisonLatencyMetrics() {
   const requestDetectionLatency = useRef(0);
   const swapComparisonLatency = useRef(0);
 
-  useEffect(() => {
+  // using useMemo here as it is the first hook to be executed
+  useMemo(() => {
     uiInitializedTime.current = new Date().getTime();
   }, []);
 
@@ -24,7 +25,7 @@ export function useDappSwapComparisonLatencyMetrics() {
       return swapComparisonLatency.current.toString();
     }
     swapComparisonLatency.current =
-      new Date().getTime() - requestDetectionLatency.current;
+      new Date().getTime() - uiInitializedTime.current;
     return swapComparisonLatency.current.toString();
   }, []);