From 9eb217074d4489e4ed95f7153dff90db74c249ea Mon Sep 17 00:00:00 2001 From: Carlton Hanna Date: Thu, 23 May 2024 10:41:58 -0600 Subject: [PATCH] `add-net-asset-values` command now correctly uses the from `flag`'s `AccAddress` (#1996) * Add from address and not from value from cmd * add change log entry * remove trailing spaces --- CHANGELOG.md | 4 ++++ x/marker/client/cli/tx.go | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 65922e010..32fdca07c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -101,6 +101,10 @@ Ref: https://keepachangelog.com/en/1.0.0/ * Rosetta has been removed from the `provenanced` executable [#1981](https://github.com/provenance-io/provenance/pull/1981). It is now a stand-alone service. See: for more info. +### Bug Fixes + +* The `add-net-asset-values` command now correctly uses the from `flag`'s `AccAddress` [#1995](https://github.com/provenance-io/provenance/issues/1995). + ### Deprecated * In the config commands, the "tendermint" and "tm" options are deprecated, replaced with "cometbft", "comet", and "cmt" [#1968](https://github.com/provenance-io/provenance/pull/1968). diff --git a/x/marker/client/cli/tx.go b/x/marker/client/cli/tx.go index dde5129fd..4a90f5965 100644 --- a/x/marker/client/cli/tx.go +++ b/x/marker/client/cli/tx.go @@ -1069,7 +1069,7 @@ func GetCmdAddNetAssetValues() *cobra.Command { return err } - return tx.GenerateOrBroadcastTxCLI(clientCtx, cmd.Flags(), types.NewMsgAddNetAssetValuesRequest(denom, clientCtx.From, netAssetValues)) + return tx.GenerateOrBroadcastTxCLI(clientCtx, cmd.Flags(), types.NewMsgAddNetAssetValuesRequest(denom, clientCtx.GetFromAddress().String(), netAssetValues)) }, } flags.AddTxFlagsToCmd(cmd)