HomePhabricator

[Cashtab] Remove parsedTxHistory from wallet state

Description

[Cashtab] Remove parsedTxHistory from wallet state

Summary:
Tx history is helpful information for the user. However, it is not necessary for any wallet actions. It should not be associated with the wallet state, especially now that we have paginated tx history which can only use the wallet state for page 1.

Remove the parsedTxHistory key from wallet state. Note we are no longer storing this key, so this change does not require a migration.

This allows us to implement more fine-grained handling of tx history and avalanche finality and achieve current performance without the hacky batchUpdates process. We could probably also lose our async queueing of websocket msgs, now that we are appropriately using Reacts setState with prev, but at the moment it works and is basically instant so will do that in another diff if it should be done.

This diff is additional prep for dropping in ecash-wallet. With wallet state parsedTxHistory no longer used by the app, we can use ecash-wallet instead of cashtab wallet state, once it has feature parity with Cashtab's custom methods.

Test Plan: npm test, npm start, send a bunch of txs, note they are instantly added to history, note if the page is full the last tx is removed from the page, note they all finalize at the same time with no issues. This diff is live at cashtab.io

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D18669

Details

Provenance
bytesofmanAuthored on Sep 23 2025, 18:44
bytesofmanPushed on Sep 24 2025, 12:07
Reviewer
Restricted Project
Differential Revision
D18669: [Cashtab] Remove parsedTxHistory from wallet state
Parents
rABC74d80d6c9b67: [e.cash] Homepage copy tweaks
Branches
Unknown
Tags
Unknown