Related to T2765. Renamed toggleHideBalance to toggleBalanceVisibility as it better describes the setting's purpose. Initialized toggleBalanceVisibility as true. Updated unit tests to reflect the change to default settings.
Details
Details
- Reviewers
bytesofman - Group Reviewers
Restricted Project - Commits
- rABC17c2034b0547: [Cashtab] Patch balance toggle initializing hidden
cd web/cashtab && npm start
using a private browser window to ensure default settings
import a wallet
observe that the balance visibility toggle is true, and that the balance is visible
toggle balance visibility off and observe that the setting persists across all screens
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- fix-balance-toggle-initializing-hidden
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 20779 Build 41220: Build Diff cashtab-tests Build 41219: arc lint + arc unit
Event Timeline
| web/cashtab/src/components/Common/Ticker.js | ||
|---|---|---|
| 88 | this is still confusing change toggleBalanceVisibility to balanceVisible | |