From a4c7faaa403896f94323f4638fbac194e143a47c Mon Sep 17 00:00:00 2001 From: Alessio Treglia Date: Sat, 20 Oct 2018 23:52:45 -0700 Subject: [PATCH] Mark --to and --amount flags required for gaiacli tx send Closes: #2547 --- PENDING.md | 1 + x/bank/client/cli/sendtx.go | 2 ++ 2 files changed, 3 insertions(+) diff --git a/PENDING.md b/PENDING.md index eda622202c..d7bcf555c0 100644 --- 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 index 8200563b0e..c72ed6fcb9 100644 --- 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 }