Page MenuHomePhabricator
Differential D17841

[ecash-herald] Improve miner parsing
ClosedPublic
Actions

Authored by bytesofman on Thu, Mar 20, 22:47.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC5b53dd1d5d01: [ecash-herald] Improve miner parsing
Summary

Stop assuming the first output is the miner for unknown miners. Parse for the miner that brought this to our attention.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
apps/
ecash-herald/
constants/
8 lines
src/
1 line
89 lines
test/
fixtures/
15 lines
82 lines

Diff 53215

View Options

apps/ecash-herald/constants/miners.ts

Loading...
View Options

apps/ecash-herald/src/events.ts

Loading...
View Options

apps/ecash-herald/src/parse.ts

Loading...
View Options

apps/ecash-herald/test/fixtures/miners.ts

Loading...
View Options

apps/ecash-herald/test/parse.test.ts

Loading...