Page MenuHomePhabricator

[Cashtab] Replace conflicting functions with more comprehensive validation function
ClosedPublic

Authored by bytesofman on Mar 2 2024, 15:04.

Details

Summary

We should only have cashtab wallet validation function. We have two because there were in the past edge cases where a wallet was "valid", as in, it would work in Cashtab, but we still wanted to migrate it, e.g. to change the format of the address.

Implement the more comprehensive validation function everywhere.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
wallet-validation-consolidate
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 27607
Build 54777: Build Diffcashtab-tests
Build 54776: arc lint + arc unit