Page MenuHomePhabricator

added total wallet balanced to saved wallets list
AbandonedPublic

Authored by kieran709 on Fri, Oct 1, 18:11.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Restricted Project
Summary

Accessing the balance of any saved wallet that contains the state object and displaying it in the saved wallets list.

Test Plan

npm start
navigate to settings page
verify wallets in saved wallets drop down display balance
verify balance is responsive

Diff Detail

Repository
rABC Bitcoin ABC
Branch
sw-balances
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16865
Build 33569: Build Diffcashtab-tests
Build 33568: arc lint + arc unit

Event Timeline

Owners added a reviewer: Restricted Owners Package.Fri, Oct 1, 18:11
bytesofman requested changes to this revision.Fri, Oct 1, 20:46

Thanks for this diff. It works but requires refinement for display aesthetics, responsiveness, and balance formatting. Since another diff is already in for this change, please abandon this rev and commence work on separate task discussed in Telegram.

Joey

This revision now requires changes to proceed.Fri, Oct 1, 20:46

Abandoned per review feedback.