feat(x/ecocredit)!: migrate add-allow-denom proposal to msg based proposal

CHANGELOG.md

@@ -9,6 +9,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### app
 
+### x/ecocredit
+
+#### API Breaking Changes
+
+- [#1342](https://github.com/regen-network/regen-ledger/pull/1342) The `NewKeeper` method in `ecocredit/marketplace` requires an `authority` address.
+- [#1342](https://github.com/regen-network/regen-ledger/pull/1342) Removed `AllowedDenom` proposal handler.
+
+#### Added
+
+- [#1337](https://github.com/regen-network/regen-ledger/pull/1342) `AddAllowedDenom` is added for msg-based gov proposal.
+
+
 #### Removed
 
 - [#1258](https://github.com/regen-network/regen-ledger/pull/1258) Remove group module from experimental config

app/app.go

@@ -100,7 +100,6 @@ import (
 	"github.com/regen-network/regen-ledger/x/ecocredit"
 	"github.com/regen-network/regen-ledger/x/ecocredit/basket"
 	ecocreditmodule "github.com/regen-network/regen-ledger/x/ecocredit/module"
-	ecoServer "github.com/regen-network/regen-ledger/x/ecocredit/server"
 
 	// unnamed import of statik for swagger UI support
 	_ "github.com/regen-network/regen-ledger/v4/client/docs/statik"
@@ -389,8 +388,6 @@ func NewRegenApp(logger log.Logger, db dbm.DB, traceStore io.Writer, loadLatest
 	}
 	app.smm.RegisterInvariants(&app.CrisisKeeper)
 
-	govRouter.AddRoute(ecocredit.RouterKey, ecoServer.NewProposalHandler(ecocreditModule.Keeper))
-
 	govConfig := govtypes.DefaultConfig()
 	app.GovKeeper = govkeeper.NewKeeper(
 		appCodec, keys[govtypes.StoreKey], app.GetSubspace(govtypes.ModuleName), app.AccountKeeper, app.BankKeeper,

proto/regen/ecocredit/marketplace/v1/tx.proto

@@ -23,6 +23,11 @@ service Msg {
 
   // BuyDirect purchases credits directly from the specified sell order.
   rpc BuyDirect(MsgBuyDirect) returns (MsgBuyDirectResponse);
+
+  // AddAllowedDenom is a governance method that allows the addition of
+  // new allowed denom.
+  // Since Revision 1
+  rpc AddAllowedDenom(MsgAddAllowedDenom) returns (MsgAddAllowedDenomResponse);
 }
 
 // MsgSell is the Msg/Sell request type.
@@ -155,3 +160,26 @@ message MsgBuyDirect {
 
 // MsgBuyDirectResponse is the Msg/BuyDirect response type.
 message MsgBuyDirectResponse {}
+
+// MsgAddAllowedDenom is the Msg/AddAllowedDenom request type.
+// Since Revision 1
+message MsgAddAllowedDenom {
+  // authority is the address of the governance account.
+  string authority = 1;
+
+  // denom is the bank denom to allow (ex. ibc/GLKHDSG423SGS)
+  string bank_denom = 2;
+
+  // display_denom is the denom to display to the user and is informational.
+  // Because the denom is likely an IBC denom, this should be chosen by
+  // governance to represent the consensus trusted name of the denom.
+  string display_denom = 3;
+
+  // exponent is the exponent that relates the denom to the display_denom and is
+  // informational
+  uint32 exponent = 4;
+}
+
+// MsgAddAllowedDenomResponse is the Msg/AddAllowedDenom response type.
+// Since Revision 1
+message MsgAddAllowedDenomResponse {}

proto/regen/ecocredit/marketplace/v1/types.proto

@@ -2,15 +2,15 @@ syntax = "proto3";
 
 package regen.ecocredit.marketplace.v1;
 
-import "gogoproto/gogo.proto";
 import "regen/ecocredit/marketplace/v1/state.proto";
 
 option go_package = "github.com/regen-network/regen-ledger/x/ecocredit/marketplace";
 
 // AllowDenomProposal is a gov Content type for approving a denom for use in the
 // marketplace.
+// Deprecated (Since Revision 1): This message is no longer used and will be removed
+// in the next version. See MsgAddAllowedDenom.
 message AllowDenomProposal {
-  option (gogoproto.goproto_stringer) = false;
 
   // title is the title of the proposal.
   string title = 1;
@@ -21,4 +21,4 @@ message AllowDenomProposal {
   // denom contains coin denom information that will be added to the
   // list of allowed denoms for use in the marketplace.
   AllowedDenom denom = 3;
-}
+}

proto/regen/ecocredit/v1/types.proto

@@ -102,6 +102,8 @@ message OriginTx {
 }
 
 // CreditTypeProposal is a gov Content type for adding a credit type.
+// Deprecated (Since Revision 1): This message is no longer used and will be removed
+// in the next version. See MsgAddCreditType.
 message CreditTypeProposal {
   option (gogoproto.goproto_stringer) = false;