Fix spending cap decoding

MetaMask · Sep 9, 2024 · b01ef5f · b01ef5f
1 parent 4027e73
commit b01ef5f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/test/e2e/helpers.js b/test/e2e/helpers.js
@@ -1077,12 +1077,6 @@ async function getEventPayloads(driver, mockedEndpoints, hasRequest = true) {
     mockedRequests.push(...(await mockedEndpoint.getSeenRequests()));
   }
 
-  console.log({
-    mockedRequests,
-    body: mockedRequests[0].body,
-    getJson: mockedRequests[0].getJson,
-  });
-
   return (
     await Promise.all(
       mockedRequests.map(async (req) => {

diff --git a/ui/pages/confirmations/components/confirm/title/title.tsx b/ui/pages/confirmations/components/confirm/title/title.tsx
@@ -23,6 +23,9 @@ import {
 } from '../../../utils';
 import { useApproveTokenSimulation } from '../info/approve/hooks/use-approve-token-simulation';
 import { useIsNFT } from '../info/approve/hooks/use-is-nft';
+import customSpendingCap from '../../custom-spending-cap';
+import { useDecodedTransactionData } from '../info/hooks/useDecodedTransactionData';
+import BigNumber from 'bignumber.js';
 
 function ConfirmBannerAlert({ ownerId }: { ownerId: string }) {
   const t = useI18nContext();
@@ -144,7 +147,13 @@ const ConfirmTitle: React.FC = memo(() => {
   const { isNFT } = useIsNFT(currentConfirmation as TransactionMeta);
 
   let customSpendingCap = '';
-  if (isTransactionMeta(currentConfirmation)) {
+  if (
+    isTransactionMeta(currentConfirmation) &&
+    [
+      TransactionType.tokenMethodApprove,
+      TransactionType.tokenMethodIncreaseAllowance,
+    ].includes(currentConfirmation.type as TransactionType)
+  ) {
     const { decimals } = useAssetDetails(
       currentConfirmation.txParams.to,
       currentConfirmation.txParams.from,