Differential D14854

[electrum] use the progress bar when broadcasting multiple transactions
ClosedPublic
Actions

Authored by PiRK on Nov 27 2023, 07:59.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC6837b86338e5: [electrum] use the progress bar when broadcasting multiple transactions

Summary

This can help the user be confident that there is progress when broadcasting multiple large consolidation transactions.

Depends on D14853

Test Plan

Prepare and broadcast a few transactions.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

broadcast_multi_tx

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 25744
Build 51068: Build Diff	electrum-tests
Build 51067: arc lint + arc unit

Event Timeline

PiRK created this revision.Nov 27 2023, 07:59

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 27 2023, 07:59

PiRK requested review of this revision.Nov 27 2023, 07:59

Harbormaster completed remote builds in B25744: Diff 43273.Nov 27 2023, 08:01

not able to arc patch this one, is this based on master?

Fabien accepted this revision.Nov 28 2023, 08:37

This revision is now accepted and ready to land.Nov 28 2023, 08:37

In D14854#333685, @bytesofman wrote:

not able to arc patch this one, is this based on master?

Sorry I thought i already landed the parent diff. Dependency added

Closed by commit rABC6837b86338e5: [electrum] use the progress bar when broadcasting multiple transactions (authored by PiRK). · Explain WhyNov 28 2023, 10:35

This revision was automatically updated to reflect the committed changes.

PiRK added a commit: rABC6837b86338e5: [electrum] use the progress bar when broadcasting multiple transactions.

Revision Contents
Changeset List

Path

Size

electrum/

electrumabc_gui/

qt/

multi_transactions_dialog.py

11 lines

Diff 43273

View Options

electrum/electrumabc_gui/qt/multi_transactions_dialog.py

[electrum] use the progress bar when broadcasting multiple transactionsClosedPublicActions