-
Notifications
You must be signed in to change notification settings - Fork 233
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
send_all_MP method - send all owned omni assets in 1 transaction #153
Comments
I have asked @nixoid to do a pull request against the spec so that it can be added |
Interesting idea. The core implementation and RPC call should be easy. ACK from my side. |
Protocol spec pull request created: |
Scratch that, just saw your PR :) |
@zathras-crypto: I'm going offline for now, but I pushed #155 as starting point. There are two items I'm not really sure about, so input/contribution would be welcomed. :) |
What should the behaviour be for reserved balances? Eg:
Should my trades & sell offers be automatically cancelled and the reserved tokens returned to my balance, followed by sending to the recipient, or should a send all only apply to "available" balances? |
For menit's ok to simply reject send all transaction if some tokens are On Wed, Jul 29, 2015 at 6:46 PM zathras-crypto [email protected]
|
@nixoid I tend to look at rejecting a transaction as a last resort, so would prefer to define the behaviour and then allow a transaction to proceed in the case of reserved balances. It's not too much trouble either way, we can either leave reserved balances in place at the original address or auto-cancel trades and then transfer reserved tokens along with the rest. Input welcomed of course :) Also FYI the implementation is taking place in #155, feel free to add any commentary/feedback. |
|
Hi there.
This is a high priority protocol feature request.
Some users have more than one omni-based asset on their address. Specifically users of Tether(http://wallet.tether.to/) already got 3 currencies + bitcoin and there are much more in plans.
We often have a need to transfer all assets from one account to another. Specifically:
It is inconvenient and expensive from technical and financial standpoint to broadcast separate transaction for each asset, especially given that wallet may support hundreds of them (holytransaction.com)
So, speaking on behalf of Tether, Holytransaction and other companies i can tell that we'd like to see this feature soon :)
Important: it has to be a protocol method(as opposed to wallet rpc command) since we build transactions with own code.
Thank you.
The text was updated successfully, but these errors were encountered: