Page MenuHomePhabricator

[ecashaddrjs] Support decoding prefixless addresses if checksum is correct
ClosedPublic

Authored by bytesofman on Mar 27 2023, 12:56.

Details

Summary

T3039

Depends on D13474

Test Plan

Review change to function and new unit test. npm test and all pass.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22729
Build 45078: Build Diff
Build 45077: arc lint + arc unit

Event Timeline

Fabien added a subscriber: Fabien.
Fabien added inline comments.
web/ecashaddrjs/src/cashaddr.js
88 ↗(On Diff #38918)

Nit: you could also print the list of supported prefixes and hint to not use a prefixless address if the prefix is not in this list

This revision is now accepted and ready to land.Mar 27 2023, 15:26

Print list of valid prefixes on prefixless error