HomePhabricator

[Cashtab] [chronik tx history p21] Better token burn parsing

Description

[Cashtab] [chronik tx history p21] Better token burn parsing

Summary:
T2447

Depends on D12158

For token transactions with high decimal count, chronik will (sometimes?) include the slpBurn field, even if the burned amount is 0...so it's not actually a burn.

This specifies that the existence of an slpBurn field is not enough to imply a token burn. Amount must be zero. This corrects an issue where non-burn txs were being parsed incorrectly (wrong amount, wrong tx type)

Test Plan:
Review new unit test and npm test
Receive an etoken tx with 9 decimal places
Verify that it is rendered in tx history as a received etoken tx with correct quantity

Reviewers: #bitcoin_abc, emack

Reviewed By: #bitcoin_abc, emack

Differential Revision: https://reviews.bitcoinabc.org/D12159

Details

Provenance
bytesofmanAuthored on Oct 5 2022, 22:35
bytesofmanPushed on Oct 6 2022, 13:49
Reviewer
Restricted Project
Differential Revision
D12159: [Cashtab] [chronik tx history p21] Better token burn parsing
Parents
rABCe4a3e2a1b2a8: [Cashtab] [chronik tx history p20] update token info cache in tx history routine
Branches
Unknown
Tags
Unknown