From d5dd4fa881abca2f723ec1be0becf6b76435fa40 Mon Sep 17 00:00:00 2001 From: Ariella Vu <20778143+digiwand@users.noreply.github.com> Date: Thu, 1 Jun 2023 18:42:19 +0200 Subject: [PATCH] ConfirmTxPage: rm redundant unapprovedTxs const (#19275) * fix typo: uncofirmedTransactions -> unconfirmedTransactions * txHelper: reorganize lines and improve test * ConfirmTx: rn unconfirmedTxs->unconfirmedTxsSorted * ConfirmTx: rn unconfirmedMessages-> unconfirmedTxs * ConfirmTxPage: rm redundant unapprovedTxs const * lint: rm unused getUnapprovedTransactions --- .../confirm-transaction/confirm-transaction.component.js | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/ui/pages/confirm-transaction/confirm-transaction.component.js b/ui/pages/confirm-transaction/confirm-transaction.component.js index 16b8d502b..9d26ed3f6 100644 --- a/ui/pages/confirm-transaction/confirm-transaction.component.js +++ b/ui/pages/confirm-transaction/confirm-transaction.component.js @@ -31,7 +31,6 @@ import { import { isTokenMethodAction } from '../../helpers/utils/transactions.util'; import { usePrevious } from '../../hooks/usePrevious'; import { - getUnapprovedTransactions, unconfirmedTransactionsListSelector, unconfirmedTransactionsHashSelector, } from '../../selectors'; @@ -56,21 +55,18 @@ const ConfirmTransaction = () => { const mostRecentOverviewPage = useSelector(getMostRecentOverviewPage); const sendTo = useSelector(getSendTo); - const unapprovedTxs = useSelector(getUnapprovedTransactions); + const unconfirmedTxsSorted = useSelector(unconfirmedTransactionsListSelector); const unconfirmedTxs = useSelector(unconfirmedTransactionsHashSelector); const totalUnapproved = unconfirmedTxsSorted.length || 0; const getTransaction = useCallback(() => { return totalUnapproved - ? unapprovedTxs[paramsTransactionId] || - unconfirmedTxs[paramsTransactionId] || - unconfirmedTxsSorted[0] + ? unconfirmedTxs[paramsTransactionId] || unconfirmedTxsSorted[0] : {}; }, [ paramsTransactionId, totalUnapproved, - unapprovedTxs, unconfirmedTxs, unconfirmedTxsSorted, ]); @@ -87,7 +83,6 @@ const ConfirmTransaction = () => { getTransaction, paramsTransactionId, totalUnapproved, - unapprovedTxs, unconfirmedTxs, unconfirmedTxsSorted, ]);