Skip to content

Commit 9e5f513

Browse files
Address PR review
1 parent e58a2f4 commit 9e5f513

File tree

6 files changed

+7
-12
lines changed

6 files changed

+7
-12
lines changed

ui/pages/confirmations/components/confirm/info/approve/approve-static-simulation/approve-static-simulation.tsx

+1-3
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,13 @@ export const ApproveStaticSimulation = () => {
2727
const { currentConfirmation: transactionMeta } =
2828
useConfirmContext<TransactionMeta>();
2929

30-
const { decimals: initialDecimals } = useAssetDetails(
30+
const { decimals } = useAssetDetails(
3131
transactionMeta?.txParams?.to,
3232
transactionMeta?.txParams?.from,
3333
transactionMeta?.txParams?.data,
3434
transactionMeta?.chainId,
3535
);
3636

37-
const decimals = initialDecimals || '0';
38-
3937
const {
4038
spendingCap,
4139
isUnlimitedSpendingCap,

ui/pages/confirmations/components/confirm/info/approve/approve.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ const ApproveInfo = () => {
3535

3636
const { spendingCap, pending } = useApproveTokenSimulation(
3737
transactionMeta,
38-
decimals || '0',
38+
decimals,
3939
);
4040

4141
const showRevokeVariant =

ui/pages/confirmations/components/confirm/info/approve/edit-spending-cap-modal/edit-spending-cap-modal.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ export const EditSpendingCapModal = ({
6464

6565
const { formattedSpendingCap, spendingCap } = useApproveTokenSimulation(
6666
transactionMeta,
67-
decimals || '0',
67+
decimals,
6868
);
6969

7070
const [customSpendingCapInputValue, setCustomSpendingCapInputValue] =

ui/pages/confirmations/components/confirm/info/approve/hooks/use-approve-token-simulation.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,15 @@ function isSpendingCapUnlimited(decodedSpendingCap: number) {
1818

1919
export const useApproveTokenSimulation = (
2020
transactionMeta: TransactionMeta,
21-
decimals: string,
21+
decimals: string | undefined,
2222
) => {
2323
const locale = useSelector(getIntlLocale);
2424
const { isNFT, pending: isNFTPending } = useIsNFT(transactionMeta);
2525
const decodedResponse = useDecodedTransactionData();
2626
const { value, pending } = decodedResponse;
2727

2828
const decodedSpendingCap = useMemo(() => {
29-
if (!value) {
29+
if (!value || !decimals) {
3030
return '0';
3131
}
3232

ui/pages/confirmations/components/confirm/info/approve/spending-cap/spending-cap.tsx

+1-4
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,7 @@ export const SpendingCap = ({
9090
Number(decimals ?? '0'),
9191
).toFixed();
9292

93-
const { pending } = useApproveTokenSimulation(
94-
transactionMeta,
95-
decimals || '0',
96-
);
93+
const { pending } = useApproveTokenSimulation(transactionMeta, decimals);
9794

9895
if (pending) {
9996
return <Container isLoading />;

ui/pages/confirmations/components/confirm/title/hooks/useCurrentSpendingCap.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ export function useCurrentSpendingCap(currentConfirmation: Confirmation) {
4343

4444
const { spendingCap, pending } = useApproveTokenSimulation(
4545
currentConfirmation as TransactionMeta,
46-
decimals || '0',
46+
decimals,
4747
);
4848

4949
let customSpendingCap = '';

0 commit comments

Comments
 (0)