Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gui: use createspend command for coin selection and missing amount #947

Conversation

jp1ac4
Copy link
Collaborator

@jp1ac4 jp1ac4 commented Jan 30, 2024

This PR is a follow-up to #927 and uses changes made in #938.

It reverts to using the createspend command in the GUI for automated coin selection and to determine the amount left to select when creating a new spend (which was previously changed in #863).

With this PR, the changes from #873 will become effective in the GUI so that (some) unconfirmed coins are used as candidates and any additional fee to pay for ancestors is included when calculating the amount left to select.

@jp1ac4 jp1ac4 force-pushed the gui-use-createspend-command-for-missing branch from 3f28d27 to cb5073c Compare January 30, 2024 13:46
@jp1ac4
Copy link
Collaborator Author

jp1ac4 commented Jan 30, 2024

Thanks to @edouardparis for spotting that the amount left to select would not update if the user deselected all coins. This was because passing an empty list of outpoints to create_spend_tx would use auto-selection rather than manual selection with an empty list of inputs. I've made a change as suggested to set the amount left to select in that case to the total destination value.

Copy link
Member

@edouardparis edouardparis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK cb5073c

@edouardparis edouardparis merged commit ae9e424 into wizardsardine:master Jan 30, 2024
18 checks passed
@jp1ac4 jp1ac4 deleted the gui-use-createspend-command-for-missing branch February 2, 2024 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants