Paths

Table of Contentst

Differential D12047

[Cashtab] Move token burn UI lock to same as other token tx method
ClosedPublic
Actions

Authored by bytesofman on Sep 23 2022, 15:48.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCe5a2c48097ad: [Cashtab] Move token burn UI lock to same as other token tx method

Summary

T2670

Cashtab locks the UI after a tx is broadcast until the utxo set has been updated. There is a bug on the burn token tx feature where the UI is unlocked manually.

This should be removed so the UI unlocks in the same way as after any other tx, by the useEffect loop that waits for balance to update.

Test Plan

npm start
Burn some eToken
Note that the UI locks and restores after utxo set is updated

Diff Detail

Repository

rABC Bitcoin ABC

Branch

patch-sendtoken-issue

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20153
Build 39991: Build Diff	cashtab-tests
Build 39990: arc lint + arc unit

Event Timeline

bytesofman created this revision.Sep 23 2022, 15:48

Herald added a reviewer: Restricted Project. · View Herald TranscriptSep 23 2022, 15:48

bytesofman requested review of this revision.Sep 23 2022, 15:48

Harbormaster completed remote builds in B20153: Diff 35157.Sep 23 2022, 15:53

emack accepted this revision.Sep 24 2022, 14:36

This revision is now accepted and ready to land.Sep 24 2022, 14:36

Closed by commit rABCe5a2c48097ad: [Cashtab] Move token burn UI lock to same as other token tx method (authored by bytesofman). · Explain WhySep 24 2022, 15:35

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCe5a2c48097ad: [Cashtab] Move token burn UI lock to same as other token tx method.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Send/

SendToken.js

1 line

Diff 35157

View Options

web/cashtab/src/components/Send/SendToken.js

[Cashtab] Move token burn UI lock to same as other token tx methodClosedPublicActions