Page MenuHomePhabricator

[Cashtab] Patch bug caused by wallet change from Tokens page
ClosedPublic

Authored by bytesofman on Mon, Apr 29, 21:33.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Commits
rABC74d647d0b05c: [Cashtab] Patch bug caused by wallet change from Tokens page
Summary

The screen defines tokens as wallet.state.tokens in one place, uses this value to do the calculations, but then triggers the useEffect on wallet.state.tokens.

Make sure the tokens used in the useEffect is the same tokens used in the dependency array.

Test Plan

npm test, npm start, navigate to tokens page, change wallets a few times, no white screen app crash

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable