Page MenuHomePhabricator

[Cashtab] Do not assume 546 sat utxo is token
AbandonedPublic

Authored by bytesofman on Aug 10 2021, 17:50.

Details

Reviewers
majcosta
Group Reviewers
Restricted Project
Summary

Cashtab assumes all 546-satoshi utxos are eToken utxos to prevent accidental token burns. However, token utxo checks are already redundant, and there is not a known instance of eToken utxos being classified as eCash utxos with the current backend.

There is still a case to be made for keeping this as a "just in case" error check. However, now that we have more users, and 5.46 XEC is the min transaction -- the app should support users testing the minimum transaction without raising the question of "why did the min transaction not show up in the receiving wallet?"

T1688

Test Plan

npm start , receive a 5.46 XEC transaction and notice that it impacts the displayed XEC balance

Diff Detail

Repository
rABC Bitcoin ABC
Branch
stop-ignoring-min-utxos
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 16299
Build 32465: Build Diffcashtab-tests
Build 32464: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Aug 10 2021, 18:01

Abandoning in favor of D9865, which still preserves this check