Page MenuHomePhabricator

[Cashtab] Remove legacy walletState parameter
ClosedPublic

Authored by bytesofman on Aug 22 2021, 22:16.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABCe4e479aeb4b3: [Cashtab] Remove legacy walletState parameter
Summary

T1742

Remove legacy walletState parameter.

walletState was replaced by wallet.state to enable local storage support for utxo set and token balances, speeding up Cashtab.

This was already done in every screen using wallet.state. This diff completes the deprecation of walletState by removing it from useWallet.js (where it was still used for incoming tx notifications)

Test Plan

npm start
Receive an XEC tx and note tx notification
Receive an eToken tx and note tx notification

Diff Detail

Repository
rABC Bitcoin ABC
Branch
remove-legacy-walletState
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16359
Build 32581: Build Diffcashtab-tests
Build 32580: arc lint + arc unit