support broadcasting multiple txns

Author: ProgramCpp

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 All notable changes will be documented here.
 
+## [v0.4.1] - TBD
+- support broadcasting multiple transactions
+- accept configs dynamically set by the user
+
 ## [v0.3.1] - 2023-05-17
 - add support for configuring fee granter for transactions
 ## [v0.3.0] - 2023-03-14

README.md

@@ -2,9 +2,8 @@
 
 A gitopia client library and cosmos sdk wrapper
 
-## Environment configs
-// TODO: accept config via parameters
-configure the below environment variables
+## configs
+configure the below config variables
 ```
 GITOPIA_ADDR
 TM_ADDR

client.go

@@ -18,7 +18,6 @@ import (
 	rtypes "github.com/gitopia/gitopia/v2/x/rewards/types"
 	"github.com/pkg/errors"
 	"github.com/sirupsen/logrus"
-	"github.com/spf13/viper"
 	rpcclient "github.com/tendermint/tendermint/rpc/client"
 	rpchttp "github.com/tendermint/tendermint/rpc/client/http"
 	ctypes "github.com/tendermint/tendermint/rpc/core/types"
@@ -51,14 +50,14 @@ func NewClient(ctx context.Context, cc client.Context, txf tx.Factory) (Client,
 	w := logger.FromContext(ctx).WriterLevel(logrus.DebugLevel)
 	cc = cc.WithOutput(w)
 
-	txf = txf.WithGasPrices(viper.GetString("GAS_PRICES")).WithGasAdjustment(GAS_ADJUSTMENT)
+	txf = txf.WithGasPrices(GAS_PRICES).WithGasAdjustment(GAS_ADJUSTMENT)
 
 	rc, err := rpchttp.New(cc.NodeURI, TM_WS_ENDPOINT)
 	if err != nil {
 		return Client{}, errors.Wrap(err, "error creating rpc client")
 	}
 
-	q, err := GetQueryClient(viper.GetString("GITOPIA_ADDR"))
+	q, err := GetQueryClient(GITOPIA_ADDR)
 	if err != nil {
 		return Client{}, errors.Wrap(err, "error creating query client")
 	}
@@ -117,9 +116,9 @@ func (c Client) AuthorizedBroadcastTx(ctx context.Context, msg sdk.Msg) error {
 	return nil
 }
 
-func (c Client) BroadcastTxAndWait(ctx context.Context, msg sdk.Msg) error {
+func (c Client) BroadcastTxAndWait(ctx context.Context, msg ...sdk.Msg) error {
 	// !!HACK!! set sequence to 0 to force refresh account sequence for every txn
-	txHash, err := BroadcastTx(c.cc, c.txf.WithSequence(0).WithFeePayer(c.Address()), msg)
+	txHash, err := BroadcastTx(c.cc, c.txf.WithSequence(0).WithFeePayer(c.Address()), msg...)
 	if err != nil {
 		return err
 	}

cmd.go

@@ -18,7 +18,6 @@ import (
 	otypes "github.com/gitopia/gitopia/v2/x/offchain/types"
 	"github.com/pkg/errors"
 	"github.com/spf13/cobra"
-	"github.com/spf13/viper"
 )
 
 func initClientConfig() {
@@ -55,8 +54,8 @@ func CommandInit(cmd *cobra.Command, appName string) error {
 
 func GetClientContext(cmd *cobra.Command) (client.Context, error) {
 	clientCtx := client.GetClientContextFromCmd(cmd)
-	clientCtx = clientCtx.WithChainID(viper.GetString("CHAIN_ID"))
-	clientCtx = clientCtx.WithNodeURI(viper.GetString("TM_ADDR"))
+	clientCtx = clientCtx.WithChainID(CHAIN_ID)
+	clientCtx = clientCtx.WithNodeURI(TM_ADDR)
 	c, err := client.NewClientFromNode(clientCtx.NodeURI)
 	if err != nil {
 		return clientCtx, errors.Wrap(err, "error creatig tm client")
@@ -74,7 +73,7 @@ func GetClientContext(cmd *cobra.Command) (client.Context, error) {
 		return clientCtx, errors.Wrap(err, "error creating keyring backend")
 	}
 	clientCtx = clientCtx.WithKeyring(kr)
-	clientCtx = clientCtx.WithKeyringDir(viper.GetString("WORKING_DIR"))
+	clientCtx = clientCtx.WithKeyringDir(WORKING_DIR)
 
 	from, err := cmd.Flags().GetString(flags.FlagFrom)
 	if err != nil {
@@ -87,7 +86,7 @@ func GetClientContext(cmd *cobra.Command) (client.Context, error) {
 
 	clientCtx = clientCtx.WithFrom(from).WithFromAddress(fromAddr).WithFromName(fromName)
 
-	feeGranterAddr := sdk.MustAccAddressFromBech32(viper.GetString("FEE_GRANTER_ADDR"))
+	feeGranterAddr := sdk.MustAccAddressFromBech32(FEE_GRANTER_ADDR)
 	clientCtx = clientCtx.WithFeeGranterAddress(feeGranterAddr)
 	return clientCtx, nil
 }

config.go

@@ -0,0 +1,33 @@
+package gitopia
+
+var APP_NAME string
+var GAS_PRICES string
+var GITOPIA_ADDR string
+var CHAIN_ID string
+var TM_ADDR string
+var WORKING_DIR string
+var FEE_GRANTER_ADDR string
+
+func WithAppName(a string) {
+	APP_NAME = a
+}
+
+func WithGasPrices(g string) {
+	GAS_PRICES = g
+}
+func WithGitopiaAddr(a string) {
+	GITOPIA_ADDR = a
+}
+
+func WithChainId(c string) {
+	CHAIN_ID = c
+}
+func WithTmAddr(a string) {
+	TM_ADDR = a
+}
+func WithWorkingDir(w string) {
+	WORKING_DIR = w
+}
+func WithFeeGranter(a string) {
+	FEE_GRANTER_ADDR = a
+}