Mark --to and --amount flags required for gaiacli tx send

Closes: #2547
cosmos · Oct 21, 2018 · a4c7faa · a4c7faa
1 parent cca2f9d
commit a4c7faa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/PENDING.md b/PENDING.md
@@ -202,6 +202,7 @@ BUG FIXES
 * Gaia CLI  (`gaiacli`)
     * [cli] [\#1997](https://github.com/cosmos/cosmos-sdk/issues/1997) Handle panics gracefully when `gaiacli stake {delegation,unbond}` fail to unmarshal delegation.
     * [cli] [\#2265](https://github.com/cosmos/cosmos-sdk/issues/2265) Fix JSON formatting of the `gaiacli send` command.
+    * [cli] [\#2547](https://github.com/cosmos/cosmos-sdk/issues/2547) Mark --to and --amount as required flags for `gaiacli tx send`.
 
 * Gaia
   * [x/stake] Return correct Tendermint validator update set on `EndBlocker` by not

diff --git a/x/bank/client/cli/sendtx.go b/x/bank/client/cli/sendtx.go
@@ -75,6 +75,8 @@ func SendTxCmd(cdc *codec.Codec) *cobra.Command {
 
 	cmd.Flags().String(flagTo, "", "Address to send coins")
 	cmd.Flags().String(flagAmount, "", "Amount of coins to send")
+	cmd.MarkFlagRequired(flagTo)
+	cmd.MarkFlagRequired(flagAmount)
 
 	return cmd
 }