fix: code-review fixes

Author: Linhead

src/hooks/queries/transaction/useBtcTransactionQuery.ts

@@ -30,7 +30,7 @@ export function useBtcTransactionQuery(
     retry: (failureCount: number): boolean => {
       // Don't retry dust amount BTC transactions
       const dustedIds = store.state.btc.dustedTransactionsIds || []
-      if (dustedIds.length > 0 && dustedIds.includes(unref(transactionId))) {
+      if (dustedIds.includes(unref(transactionId))) {
         return false
       }
       return retryFactory(Cryptos.BTC, unref(transactionId))(failureCount)

src/hooks/queries/transaction/useDashTransactionQuery.ts

@@ -30,7 +30,7 @@ export function useDashTransactionQuery(
     retry: (failureCount: number): boolean => {
       // Don't retry dust amount DASH transactions
       const dustedIds = store.state.dash.dustedTransactionsIds || []
-      if (dustedIds.length > 0 && dustedIds.includes(unref(transactionId))) {
+      if (dustedIds.includes(unref(transactionId))) {
         return false
       }
       return retryFactory(Cryptos.DASH, unref(transactionId))(failureCount)

src/hooks/queries/transaction/useDogeTransactionQuery.ts

@@ -30,7 +30,7 @@ export function useDogeTransactionQuery(
     retry: (failureCount: number): boolean => {
       // Don't retry dust amount DOGE transactions
       const dustedIds = store.state.doge.dustedTransactionsIds || []
-      if (dustedIds.length > 0 && dustedIds.includes(unref(transactionId))) {
+      if (dustedIds.includes(unref(transactionId))) {
         return false
       }
       return retryFactory(Cryptos.DOGE, unref(transactionId))(failureCount)

src/store/modules/btc-base/btc-base-actions.js

@@ -231,15 +231,11 @@ function createActions(options) {
 
         return hash
       } catch (error) {
-if (error.response?.data?.includes?.('dust') ||
-  error.response?.data?.error?.message?.includes?.('dust')) {
-  context.commit('dustedTransactionsIds', signedTransaction.txid);
-}
-          if (
-            ((crypto === 'BTC' || crypto === 'DOGE') && error.response.data.includes('dust')) ||
-            (crypto === 'DASH' && error.response.data.error.message.includes('dust'))
-          )
-            context.commit('dustedTransactionsIds', signedTransaction.txid)
+        if (
+          error.response?.data?.includes('dust') ||
+          error.response?.data?.error?.message?.includes('dust')
+        ) {
+          context.commit('addDustedTransactionId', signedTransaction.txid)
         }
         context.commit('transactions', [{ hash: signedTransaction.txid, status: 'REJECTED' }])
 

src/store/modules/btc-base/btc-base-mutations.js

@@ -44,7 +44,7 @@ export default (initialState) => ({
     })
   },
 
-  addDustedTransactionsId(state, dustedTransactionId) {
+  addDustedTransactionId(state, dustedTransactionId) {
     state.dustedTransactionsIds.push(dustedTransactionId)
   },