Page MenuHomePhabricator
Differential D18770

[electrum] exclude already added utxos when adding more wallet UTXOs to a proof
ClosedPublic
Actions

Authored by PiRK on Oct 8 2025, 13:01.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC3fb416a982d4: [electrum] exclude already added utxos when adding more wallet UTXOs to a proof
Summary

When adding additional stakes from the wallet to an existing proof, it is confusing to have to choose from a list that contains already added stakes.
Filter the list accordingly.

Test Plan

Select a few coins, right click, Build Avalanche Proof, click "add coins from wallet" button and check that the list now no longer has the coins already in the proof.

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

PiRK created this revision.Oct 8 2025, 13:01
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 8 2025, 13:01
PiRK requested review of this revision.Oct 8 2025, 13:01
PiRK added a child revision: D18771: [electrum] rename Stake.utxo to Stake.outpoint.Oct 8 2025, 13:03
PiRK updated this revision to Diff 56104.Oct 8 2025, 13:07

don't break python 3.9 support

Harbormaster completed remote builds in B34735: Diff 56102.Oct 8 2025, 13:08
PiRK added inline comments.Oct 8 2025, 13:09
electrum/electrumabc_gui/qt/avalanche/proof_editor.py
545 ↗(On Diff #56102)

from transaction.py

class OutPoint:
    ...   
    def __str__(self):
        return f"{self.txid.to_string()}:{self.n}"
Harbormaster completed remote builds in B34737: Diff 56104.Oct 8 2025, 13:15
Fabien accepted this revision.Oct 8 2025, 14:36
This revision is now accepted and ready to land.Oct 8 2025, 14:36
Closed by commit rABC3fb416a982d4: [electrum] exclude already added utxos when adding more wallet UTXOs to a proof (authored by PiRK). · Explain WhyOct 8 2025, 14:54
This revision was automatically updated to reflect the committed changes.
PiRK added a commit: rABC3fb416a982d4: [electrum] exclude already added utxos when adding more wallet UTXOs to a proof.

Revision Contents
Changeset List

PathSize
electrum/
electrumabc_gui/
qt/
avalanche/
17 lines

Diff 56113

View Options

electrum/electrumabc_gui/qt/avalanche/proof_editor.py

Loading...