-
Notifications
You must be signed in to change notification settings - Fork 719
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
[Trivial] [RPC] Fix listcoldutxos help text #1753
[Trivial] [RPC] Fix listcoldutxos help text #1753
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 611a30a with a minor nit.
" \"confirmations\" : n (numeric) The number of confirmations of the P2CS utxo\n" | ||
" \"cold-staker\" : \"address\" (string) The cold-staker address of the P2CS utxo\n" | ||
" \"coin-owner\" : \"address\" (string) The coin-owner address of the P2CS utxo\n" | ||
" \"whitelisted\" : \"true\" (boolean) \"true\"/\"false\" coin-owner in delegator whitelist\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor nit: true|false
instead of only true
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well... guess we can merge this as is for now. utACK 611a30a
Some of the example "Results" in the help command of
listcoldutxos
were misleading, such as "whitelisted" showing as a string instead of a boolean, cold-staker/owner showing as "n", often associated with a numerical value, and "txidn" showing as a "string" even though it's the index number of the UTXO - This PR simply corrects them to what they should be.