Skip to content

qt: Add sweep private key dialog#297

Open
kwsantiago wants to merge 2 commits intobitcoinknots:29.x-knotsfrom
privkeyio:gui-sweep-privkey
Open

qt: Add sweep private key dialog#297
kwsantiago wants to merge 2 commits intobitcoinknots:29.x-knotsfrom
privkeyio:gui-sweep-privkey

Conversation

@kwsantiago
Copy link
Copy Markdown

Summary

  • Adds a GUI dialog for sweeping coins from WIF private keys into the active wallet
  • Accessible via File > "Sweep private key..." (same pattern as Sign/Verify Message)
  • Wraps the existing sweepprivkeys RPC; supports multiple keys and optional label

Rework of bitcoin-core/gui#650 per review feedback: import private key is a footgun, only sweep should be exposed in the GUI.

Dialog that sweeps coins from WIF private key(s) into the
active wallet by calling the sweepprivkeys RPC. Supports
multiple keys (one per line) and an optional label.

RPC runs on a background thread to avoid blocking the GUI.
Private key text is scrubbed from memory on clear and close.
Add "Sweep private key..." action to the File menu, following
the same pattern as Sign/Verify Message.
@luke-jr
Copy link
Copy Markdown
Collaborator

luke-jr commented Apr 14, 2026

Probably want a wallet chooser (when multiple wallets open)

luke-jr pushed a commit that referenced this pull request Apr 14, 2026
Dialog that sweeps coins from WIF private key(s) into the
active wallet by calling the sweepprivkeys RPC. Supports
multiple keys (one per line) and an optional label.

RPC runs on a background thread to avoid blocking the GUI.
Private key text is scrubbed from memory on clear and close.

Github-Pull: #297
Rebased-From: e01da56
luke-jr pushed a commit that referenced this pull request Apr 14, 2026
Add "Sweep private key..." action to the File menu, following
the same pattern as Sign/Verify Message.

Github-Pull: #297
Rebased-From: 804d4d1
@luke-jr luke-jr added this to the 29.3 (2nd) milestone Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants