-
Notifications
You must be signed in to change notification settings - Fork 172
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
Allow an RPC style coin control options as well #1260
Comments
I think this is a great idea and is very complimentary to the stake splitter/side staking work I am doing! |
If those rpc inputs were JSON formatted, this could be nicely implemented. |
This could easily become a giant command with many arguments and crazy formatting. |
I was thinking 3 arguments 1 optional. 2 objects 1 string. String is optional change address specified by user |
Note that consolidateunspent partially addresses this issue. |
Kind of a thinking out of the box style from #1240. I've recently done some coding in these areas so I have understanding etc on this. Currently any coin control ability is mainly done via GUI in ease for users. When it comes to RPC you can do utxos in a createrawtransaction, etc which can be challenging and not-error free for even more advanced users and if not handled correctly change can be burned as a miner fee.
I propose some more gridcoin based RPC commands to allow usage in a more user friendly way. perhaps since @denravonska wants more separation of gridcoin code from main files this could be done in a
utxo.cpp/h
I'd like to assign myself this task of course if there is support for such functionality.
Propose to add:
I've done work in unspent area, also have made a custom rpc for burning dust utxos as well and of course the changing of the sendmany to use no account specified. so I have the understanding and know how to get this moving along. With these kind of commands it would make things more at ease for users instead of resorting to a manual raw transaction creation.
I'd like more user friendly options available to the common user.
The text was updated successfully, but these errors were encountered: