Page MenuHomePhabricator

[Cashtab] Remove legacy address display
ClosedPublic

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

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Commits
rABCd94007b876b3: [Cashtab] Remove legacy address display
Summary

T1730

Remove option for user to toggle between ecash: prefixes and legacy bitcoincash: prefixes. This diff also shows the prefix and includes some css adjustments for fitting the longer address string.

Initially I attempted to deprecate all support for legacy prefixes, but this requires some additional attention in functions and unit tests and hence will be done in a different diff.

Test Plan

npm start, check the appearance of address on home screen, toggle to eToken, check appearance, note any aesthetic complaints or responsiveness issues

Diff Detail

Repository
rABC Bitcoin ABC
Branch
remove-bch-slp-addr-support
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16331
Build 32525: Build Diffcashtab-tests
Build 32524: arc lint + arc unit