search

sumoprojects / SumoGUIWallet

Sumokoin GUI Wallet
Other
38 stars 46 forks source link

Sweep all #19

Closed XzenTorXz closed 6 years ago

XzenTorXz commented 6 years ago

Hey I forked your awesome wallet for Electroneum!

You might be interessed in my sweep_all implementation. If I understand it correctly the button for "all coins" in a transfer will always fail, because of the missing transaction fees. So I decide using the sweep_all method if the transfered balance equals the unlocked balance. The user needs to confirm that he wants to transfer all the money at the time the tranaction is made. If your interested I can make a pull-request.

The main commit: https://github.com/XzenTorXz/ElectroneumGUIWallet/commit/c1a6a588ad4029d92b817644632d11beac91c370 (theres still an debug alert and the final tranaction message throws an error, those are fixed in alter commits)

billaue2 commented 6 years ago

Thanks. I think it's a good idea.

XzenTorXz commented 6 years ago

i created a pull request

sumoprojects commented 6 years ago