Fix bug with relevant changedTxs set

samholmes · samholmes · commit feb412961665 · 2025-02-10T15:32:57.000-08:00
Only transactions that have been streamed are relevant for the
changedTxs set because we _always_ stomp the changed tx onto the
streamedTxs array.
diff --git a/src/hooks/useTransactionList.ts b/src/hooks/useTransactionList.ts
@@ -85,7 +85,7 @@ export function useTransactionList(wallet: EdgeCurrencyWallet, tokenId: EdgeToke
     const cleanupChanged = wallet.on('transactionsChanged', txs => {
       let relevant = false
       for (const tx of txs) {
-        if (tx.tokenId === tokenId) {
+        if (tx.tokenId === tokenId && streamedTxs.some(streamedTx => streamedTx.txid === tx.txid)) {
           relevant = true
           changedTxs.set(tx.txid, tx)
         }

Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ export function useTransactionList(wallet: EdgeCurrencyWallet, tokenId: EdgeToke`
`85`	`85`	`const cleanupChanged = wallet.on('transactionsChanged', txs => {`
`86`	`86`	`let relevant = false`
`87`	`87`	`for (const tx of txs) {`
`88`		`- if (tx.tokenId === tokenId) {`
	`88`	`+ if (tx.tokenId === tokenId && streamedTxs.some(streamedTx => streamedTx.txid === tx.txid)) {`
`89`	`89`	`relevant = true`
`90`	`90`	`changedTxs.set(tx.txid, tx)`
`91`	`91`	`}`