[ecashaddrjs] Support ecregtest prefix
ClosedPublic
Actions

Authored by bytesofman on Apr 11 2023, 14:03.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix

Summary

T3106

Add ecregtest as a valid prefix. Do not add it to VALID_PREFIXES_MAINNET

Test Plan

I did not add any specific unit tests for this prefix (similar to bchreg prefix). However, npm test to confirm existing functionality is not impacted.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Apr 11 2023, 14:03

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 11 2023, 14:03

bytesofman requested review of this revision.Apr 11 2023, 14:03

Harbormaster completed remote builds in B23199: Diff 39543.Apr 11 2023, 14:03

Fabien accepted this revision.Apr 11 2023, 14:17

This revision is now accepted and ready to land.Apr 11 2023, 14:17

Closed by commit rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix (authored by bytesofman). · Explain WhyApr 11 2023, 15:48

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCdbd84c022d0c: [ecashaddrjs] Support ecregtest prefix.

Revision Contents
Changeset List

Path

Size

modules/

ecashaddrjs/

src/

cashaddr.js

1 line

Diff 39550

View Options

modules/ecashaddrjs/src/cashaddr.js

[ecashaddrjs] Support ecregtest prefixClosedPublicActions