address comments

proto/axelar/reward/v1beta1/query.proto

@@ -9,7 +9,8 @@ import "axelar/reward/v1beta1/params.proto";
 option (gogoproto.goproto_getters_all) = false;
 
 // InflationRateRequest represents a message that queries the Axelar specific
-// inflation RPC method.
+// inflation RPC method. string is used for the validator address instead of
+// bytes so that it's supported as an arg in the generated REST API
 message InflationRateRequest { string validator = 1; }
 
 message InflationRateResponse {

x/reward/client/cli/query.go

@@ -46,10 +46,8 @@ func GetCmdInflationRate() *cobra.Command {
 			return err
 		}
 
-		if *validator != "" {
-			if _, err := sdk.ValAddressFromBech32(*validator); err != nil {
-				return sdkerrors.Wrap(err, "invalid validator address")
-			}
+		if _, err := sdk.ValAddressFromBech32(*validator); *validator != "" && err != nil {
+			return sdkerrors.Wrap(err, "invalid validator address")
 		}
 
 		queryClient := types.NewQueryServiceClient(clientCtx)

x/reward/keeper/grpc_query.go

@@ -38,8 +38,6 @@ func (q Querier) InflationRate(c context.Context, req *types.InflationRateReques
 	keyManagementInflation := params.KeyMgmtRelativeInflationRate.Mul(baseInflation)
 
 	validator := sdk.ValAddress{}
-
-	// Get network inflation rate if no validator is specified
 	if req.Validator != "" {
 		var err error
 
@@ -55,6 +53,8 @@ func (q Querier) InflationRate(c context.Context, req *types.InflationRateReques
 		}
 
 		maintainers := q.nexus.GetChainMaintainers(ctx, chain)
+
+		// If no validator is specified, check if there are any maintainers to get the max network inflation rate
 		if len(validator) == 0 {
 			return len(maintainers) > 0
 		}