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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 43283

View Options

electrum/electrumabc_gui/qt/multi_transactions_dialog.py

[electrum] use the progress bar when broadcasting multiple transactionsClosedPublicActions