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

Add daemon commands to manage UTXOs individually. #1173

Closed
Nethlek opened this issue Jun 19, 2018 · 17 comments
Closed

Add daemon commands to manage UTXOs individually. #1173

Nethlek opened this issue Jun 19, 2018 · 17 comments

Comments

@Nethlek
Copy link

Nethlek commented Jun 19, 2018

for example, assign each of them a number based on age and add a command to list all utxos either by age, address or balance (or an option for each). then add another command to select individual utxos for payments, eg 'sendutxo fromutxo# togridcoinaddress amount customchangeaddress' with the last one being optional.

@Nethlek
Copy link
Author

Nethlek commented Jun 20, 2018

im also offering a reward of 1000 grc to whoever adds this functionality (including a simple way to use custom change addresses).

@personthingman2
Copy link
Contributor

You can view your UTXOs with listunspent. It appears to sort them alphabetically by TXID, which doesn't make much sense to me. It would make more sense to sort them by size or number of confirmations.

And you can individually manage UTXOs with createrawtransaction but that's no fun and can be dangerous if you don't know what you're doing. I would appreciate a easier to use command for managing UTXOs.

@Nethlek
Copy link
Author

Nethlek commented Jun 20, 2018

i know about the raw transaction thing, the point is that its too cumbersome to deal with on a regular basis. if instead i could just enter eg 'sendutxo 3 14 22 myaddress amount customchangeaddress' it would be a lot easier to manage. i know i can achieve the same result already it would just be a pita to manage 200 utxos this way. im offering the 1k specifically for making a quick and easy to use command for it. i'd be extra happy if it supported using labels for addresses too if it doesnt do that already, i didnt see anything about that on the list of commands though.

@jamescowens
Copy link
Member

I think Nethlek’s idea is a really good one.

@barton2526
Copy link
Member

barton2526 commented Jun 27, 2018

@denravonska @TheCharlatan @jamescowens I have received the bounty funds from @Nethlek and will be acting as an escrow for this bounty. The address containing the bounty is S3bH2R3n2meoMaAwNUmBVqmR5eVVDT5v3m

So far the address contains:
2000 GRC (1000 from Nethlek, 500 from @jamescowens, and 500 from myself)

If anyone else would like to donate to the bounty please send funds to that address and I will update this post.

@jamescowens
Copy link
Member

I just added 500 GRC to it. So it is now at 2000 GRC.

@Nethlek
Copy link
Author

Nethlek commented Aug 2, 2018

if no one objects, the bounty (my 1k that is) may also be payed out for implementing custom staking addresses instead, as recently discussed on slack. in short, the implementation of a feature that allows staking rewards to be forwarded to a different address while keeping the original amount at the same address.

@Nethlek
Copy link
Author

Nethlek commented Aug 3, 2018

i forgot to mention, this only applies if it can be setup using the daemon or by editing the config.

@iFoggz
Copy link
Member

iFoggz commented Aug 11, 2018

a sendutxoto does have some quirks to address. i would think the syntax would be something along the lines of:

`sendutxo '{"txid": vout, "txid": vout }' '{"address": amount, "address": amount }'

CreateTransaction can probably do this as well but handling this much like coin control does for inputs.

i cannot see a sendutxo 3, 5, 7, 9 etc being viable since unspent transactions u need the txid and vout anyway and those numbers woudl change as they are used and require another iteration to re look them up

@jamescowens
Copy link
Member

If we extend my stake splitter to redirect rewards, it replaces the need for this altogether I think... @Nethlek agrees...

@iFoggz
Copy link
Member

iFoggz commented Aug 11, 2018

I think even with your stake splitter to redirect rewards this could also be a good tool for coin control for the daemon much like the gui as well. so this should also be a rpc tool as well. currently you can only select an account or send based on what the wallet chooses via rpc

@jamescowens
Copy link
Member

You are right... it is actually harder to code than the stake splitting stuff though!! :)

@iFoggz
Copy link
Member

iFoggz commented Aug 11, 2018

I actually have done some coding like a utxo dust burner etc. so I actually know my way around this so i'd separate my idea on top of this to another issue/pr as i know i can make this possible. i was just thinking beyond the box of this current idea

@jamescowens
Copy link
Member

jamescowens commented Aug 12, 2018 via email

@iFoggz
Copy link
Member

iFoggz commented Aug 12, 2018

yea i made an issue. and if any one thinks of other style commands then i can implement them if support for them to be added is founded

@barton2526
Copy link
Member

After consulting with the bounty issuers (Nethlek, myself, and Jim), we have come to agreement that this bounty is complete and should be paid out. I have sent 2000 GRC plus 30 GRC from 3 staked blocks of interest on the bounty address to an address designated by Jim (as he submitted the PR to solve the issue). TXID can be found at https://gridcoinstats.eu/block/1521254/08f12ade0510fb5d4415298c0cf295b828e0332722e0020e0c05e3c2bdd14960

@barton2526
Copy link
Member

@denravonska This issue can now be closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants