Skip to content

Commit

Permalink
feat: Conditionally disable nonce editing when smart transactions are…
Browse files Browse the repository at this point in the history
… enabled
  • Loading branch information
pedronfigueiredo committed Jan 24, 2025
1 parent 6a50aba commit cff8511
Showing 1 changed file with 9 additions and 4 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { TransactionMeta } from '@metamask/transaction-controller';
import React, { useEffect } from 'react';
import { useDispatch, useSelector } from 'react-redux';
import { TransactionMeta } from '@metamask/transaction-controller';

import { getSmartTransactionsPreferenceEnabled } from '../../../../../../../../shared/modules/selectors';
import {
ConfirmInfoRow,
ConfirmInfoRowText,
Expand All @@ -17,8 +17,8 @@ import {
showModal,
updateCustomNonce,
} from '../../../../../../../store/actions';
import { selectConfirmationAdvancedDetailsOpen } from '../../../../../selectors/preferences';
import { useConfirmContext } from '../../../../../context/confirm';
import { selectConfirmationAdvancedDetailsOpen } from '../../../../../selectors/preferences';
import { isSignatureTransactionType } from '../../../../../utils';
import { TransactionData } from '../transaction-data/transaction-data';

Expand Down Expand Up @@ -53,6 +53,9 @@ const NonceDetails = () => {
);

const displayedNonce = customNonceValue || nextNonce;
const isSmartTransactionsEnabled = useSelector(
getSmartTransactionsPreferenceEnabled,
);

return (
<ConfirmInfoSection data-testid="advanced-details-nonce-section">
Expand All @@ -63,7 +66,9 @@ const NonceDetails = () => {
<ConfirmInfoRowText
data-testid="advanced-details-displayed-nonce"
text={`${displayedNonce}`}
onEditClick={() => openEditNonceModal()}
onEditClick={
isSmartTransactionsEnabled ? undefined : () => openEditNonceModal()
}
editIconClassName="edit-nonce-btn"
editIconDataTestId="edit-nonce-icon"
/>
Expand Down

0 comments on commit cff8511

Please sign in to comment.