Page MenuHomePhabricator

[Cashtab] Parse token txs in tx history
ClosedPublic

Authored by bytesofman on Mar 24 2021, 19:12.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC4ae0093b9d1b: [Cashtab] Parse token txs in tx history
Summary

Parse and render information about token transctions for tx history

Test Plan

npm start, observe token txs in tx history, check at different widths for responsiveness, confirm rendered info matches explorer output by clicking the tx to view at explorer.be.cash

Diff Detail

Repository
rABC Bitcoin ABC
Branch
txhistory-parse-tokens
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 15402
Build 30727: Build Diffcashtab-tests
Build 30726: arc lint + arc unit

Event Timeline

Fabien requested changes to this revision.Mar 26 2021, 19:22
Fabien added a subscriber: Fabien.
Fabien added inline comments.
web/cashtab/src/hooks/useBCH.js
197

remove

207

The token parsing deserves some unit tests

web/cashtab/src/hooks/useWallet.js
203

dito

205

dito

This revision now requires changes to proceed.Mar 26 2021, 19:22
bytesofman marked 3 inline comments as done.

Add unit tests with mocks for parsing token txs, limit rendered tx count to 5 (api load evident in testing), set rendered tx count as a parameter in Ticker.js

This revision is now accepted and ready to land.Mar 27 2021, 15:50