-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
desktopkeep-desktop GUI applicationkeep-desktop GUI applicationenhancementNew feature or requestNew feature or requestmediump2PriorityPriority
Description
Problem
The desktop wallet screen (keep-desktop/src/screen/wallet.rs) has complete UI/view layers for descriptor coordination — share selection, network picker, recovery tier configuration, progress display. But the message handlers are not wired into the main event loop, so the coordination flow doesn't actually work end-to-end.
Solution
Connect the wallet screen's coordination flow to KfpNode via the existing app message system.
What exists
WalletScreenwithSetupStateconfiguration state machineDescriptorProgressenum tracking coordination phasesview_configure()for policy editingview_coordinating()for progress displaywallet_card()for listing completed descriptorsderive_xpub()helper infrost.rs
What's missing
- Message handler in
app.rsto dispatchWalletScreenmessages toKfpNode KfpNodeEvent→Messagemapping for descriptor events (Proposed, Contributed, Complete, Failed)- Progress state updates as contributions/ACKs arrive
- Error handling and timeout display
- Store completed descriptor via
keep.store_wallet_descriptor()
Dependencies
- Should include XpubAnnounce flow (Add XpubAnnounce pre-exchange for recovery keys #269) and key proof PSBT (Add key proof PSBT to DescriptorAck #270) for the full enhanced flow
- Follow the same handler wiring pattern used for FROST signing in the network screen
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
desktopkeep-desktop GUI applicationkeep-desktop GUI applicationenhancementNew feature or requestNew feature or requestmediump2PriorityPriority