Page MenuHomePhabricator

[ecash-herald] Show less info in daily msgs
ClosedPublic

Authored by bytesofman on Wed, Nov 20, 23:49.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC71cad5861890: [ecash-herald] Show less info in daily msgs
Summary

Need to keep track of "impact" in updates here. While the detail is useful impact for devs and power users -- e.g., changes in Agora volume are a good KPI to watch for the impact of this feature -- we need to cut out as much info as possible for the updates to be interesting to users. This will also support plans to prep these daily summaries for twitter.

Reduce the number of tokens we show. Do not show invalid token txs.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
herald-tighten-dailies
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 31240
Build 61979: Build Diffecash-herald-tests
Build 61978: arc lint + arc unit