Paths

Table of Contentst

Differential D13402

[alias-server] Remove toLowerCase filter
AbandonedPublic
Actions

Authored by bytesofman on Mar 21 2023, 16:18.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Summary

T3007

Depends on D13398

Now that the valid character set for aliases is restricted, toLowerCase() is obsolete. Delete this in function and delete unit tests associated with this functionality.

Test Plan

npm test
npm run live

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22774
Build 45168: Build Diff
Build 45167: arc lint + arc unit

Event Timeline

bytesofman created this revision.Mar 21 2023, 16:18

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 21 2023, 16:18

bytesofman requested review of this revision.Mar 21 2023, 16:18

Harbormaster completed remote builds in B22566: Diff 38735.Mar 21 2023, 16:18

bytesofman added a child revision: D13403: [alias-server] add server state api endpoint.Mar 21 2023, 16:29

rebase

Harbormaster completed remote builds in B22757: Diff 38963.Mar 27 2023, 21:32

rebase

Harbormaster completed remote builds in B22774: Diff 38980.Mar 27 2023, 23:19

emack accepted this revision.Mar 29 2023, 00:33

This revision is now accepted and ready to land.Mar 29 2023, 00:33

bytesofman abandoned this revision.Mar 30 2023, 15:49

Revision Contents
Changeset List

Path

Size

web/

alias-server/

alias.js

7 lines

test/

alias.js

6 lines

mocks/

aliasMocks.js

28 lines

Diff 38980

View Options

web/alias-server/alias.js

View Options

web/alias-server/test/alias.js

View Options

web/alias-server/test/mocks/aliasMocks.js

[alias-server] Remove toLowerCase filterAbandonedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 38980

web/alias-server/alias.js

web/alias-server/test/alias.js

web/alias-server/test/mocks/aliasMocks.js

[alias-server] Remove toLowerCase filter
AbandonedPublic
Actions

Revision Contents
Changeset List