CLI Multiple Coins

[rigelrozanski]

This issue is intended to be a discussion space surrounding sending transaction coins through the CLI. Currently there are three transaction fields using coins:

• coin: string identifier for the coin type
• fee: integer containing the fee
• amount: integer containing the amount to send in the transaction

Problems with this current setup:

• only allows for one coin type for the fee
• only allows for one coin type for the transaction amount
• type of coin must be the same for the transaction as well as the fee

Initial proposed solution: hack it with slashes. Rather than having fee and amount be integer types, we could make them strings which we parse at execution, this would of course limit the use of the slash character within the coin-type name if operating from the CLI. This would also eliminate the need for the coin flag, as it would be built right into fee or amount. Here are some examples that could be valid strings to send into the proposed fee or amount flags:

blank/1/btc/2
gold/10
atoms/10/btc/100/silver/666
etc.

The intention of using this is to simplify use of the CLI so the user doesn't need to point to a .json file nor type any json code. Thoughts?

[ebuchman]

what about this as a format: atoms:10,btc:100,silver:666

[mappum]

What if we even limit the coin names not to include numbers so we could do
this: 10atom,100btc,666silver. Add we could support spaces so if you
quote your CLI arg you could do this: "10 ATOM, 100 BTC".

[ebuchman]

Great idea!

…

On Thu, Feb 9, 2017 at 5:21 PM, ᴍᴀᴛᴛ ʙᴇʟʟ ***@***.***> wrote: What if we even limit the coin names not to include numbers so we could do this: 10atom,100btc,666silver. Add we could support spaces so if you quote your CLI arg you could do this: "10 ATOM, 100 BTC". — You are receiving this because you commented. Reply to this email directly, view it on GitHub <https://github.com/tendermint/basecoin/issues/24#issuecomment-278793279>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACMb70G5IuGQzkhCz8rKX0HuRhYxZKZ4ks5ra5FNgaJpZM4L8sv-> .