Skip to content

Commit

Permalink
fix: add missing signers in proto message (#2095)
Browse files Browse the repository at this point in the history
* add missing signers in msgs

* add changelog entries

* nit
  • Loading branch information
sainoe committed Jul 25, 2024
1 parent 7b419f1 commit aedc0c8
Show file tree
Hide file tree
Showing 4 changed files with 163 additions and 108 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Fix incorrect message defitions in the proto files of the provider module
([\#2095](https://github.com/cosmos/interchain-security/pull/2095))
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
- Fix incorrect message defitions in the proto files of the provider module
([\#2095](https://github.com/cosmos/interchain-security/pull/2095))
6 changes: 6 additions & 0 deletions proto/interchain_security/ccv/provider/v1/tx.proto
Original file line number Diff line number Diff line change
Expand Up @@ -270,6 +270,8 @@ message MsgOptOutResponse {}
message MsgSetConsumerCommissionRate {
option (gogoproto.equal) = false;
option (gogoproto.goproto_getters) = false;
option (cosmos.msg.v1.signer) = "signer";

// The validator address on the provider
string provider_addr = 1 [ (gogoproto.moretags) = "yaml:\"address\"" ];
// The chain id of the consumer chain to set a commission rate
Expand All @@ -281,6 +283,8 @@ message MsgSetConsumerCommissionRate {
(gogoproto.customtype) = "cosmossdk.io/math.LegacyDec",
(gogoproto.nullable) = false
];
// signer address
string signer = 4 [(cosmos_proto.scalar) = "cosmos.AddressString"];
}


Expand All @@ -292,6 +296,8 @@ message MsgSetConsumerCommissionRateResponse {}
//
// Note: this replaces ConsumerModificationProposal which is deprecated and will be removed soon
message MsgConsumerModification {
option (cosmos.msg.v1.signer) = "authority";

// the title of the proposal
string title = 1;
// the description of the proposal
Expand Down
Loading

0 comments on commit aedc0c8

Please sign in to comment.