Merge branch 'main' into dwedul/1760-marker-bank

.github/workflows/test.yml

@@ -53,7 +53,6 @@ jobs:
         run: |
           grep -vF \
             -e 'github.com/provenance-io/provenance/cmd/provenanced/cmd' \
-            -e 'github.com/provenance-io/provenance/internal/antewrapper' \
             -e 'github.com/provenance-io/provenance/x/ibchooks' \
             -e 'github.com/provenance-io/provenance/x/ibcratelimit/module' \
             -e 'github.com/provenance-io/provenance/x/ibcratelimit/simulation' \
@@ -62,7 +61,6 @@ jobs:
           split -d -n l/3 pkgs.txt.tmp pkgs.txt.part.
           printf '%s\n' \
             'github.com/provenance-io/provenance/cmd/provenanced/cmd' \
-            'github.com/provenance-io/provenance/internal/antewrapper' \
             'github.com/provenance-io/provenance/x/ibchooks' \
             'github.com/provenance-io/provenance/x/ibcratelimit/module' \
             'github.com/provenance-io/provenance/x/ibcratelimit/simulation' \

CHANGELOG.md

@@ -73,6 +73,10 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * Removes sync-info code for sdk v0.50 [#1760](https://github.com/provenance-io/provenance/issues/1760).
 * Fix most of the failing unit tests [#1943](https://github.com/provenance-io/provenance/pull/1943).
 * Clean up ReadFromClient [#1760](https://github.com/provenance-io/provenance/issues/1760).
+* Remove all `GetSigners()` methods [#1957](https://github.com/provenance-io/provenance/pull/1957).
+* Ensure all `Msg`s have correctly identified `signer` fields [#1957](https://github.com/provenance-io/provenance/pull/1957).
+* Clean up all the module codecs [#1957](https://github.com/provenance-io/provenance/pull/1957).
+* Switch to auto-generated `String` and `Equal` methods for most proto messages [#1957](https://github.com/provenance-io/provenance/pull/1957).
 * Clean up the marker module's expected BankKeeper interface [#1954](https://github.com/provenance-io/provenance/pull/1954).
 
 ### Dependencies

client/docs/swagger-ui/swagger.yaml

@@ -69847,43 +69847,50 @@ paths:
                   properties:
                     msg_type_url:
                       type: string
+                      description: >-
+                        msg_type_url is the type-url of the message with the
+                        added fee, e.g. "/cosmos.bank.v1beta1.MsgSend".
                     additional_fee:
-                      title: >-
-                        additional_fee can pay in any Coin( basically a Denom
-                        and Amount, Amount can be zero)
+                      description: >-
+                        additional_fee is the extra fee that is required for the
+                        given message type (can be in any denom).
                       type: object
                       properties:
                         denom:
                           type: string
                         amount:
                           type: string
-                      description: >-
-                        Coin defines a token with a denomination and an amount.
-
-
-                        NOTE: The amount field is an Int which implements the
-                        custom method
-
-                        signatures required by gogoproto.
                     recipient:
                       type: string
+                      description: >-
+                        recipient is an option address that will receive a
+                        portion of the additional fee.
+
+                        There can only be a recipient if the
+                        recipient_basis_points is not zero.
                     recipient_basis_points:
                       type: integer
                       format: int64
-                  title: >-
-                    MsgFee is the core of what gets stored on the blockchain
+                      description: >-
+                        recipient_basis_points is an optional portion of the
+                        additional fee to be sent to the recipient.
 
-                    it consists of four parts
+                        Must be between 0 and 10,000 (inclusive).
 
-                    1. the msg type url, i.e. /cosmos.bank.v1beta1.MsgSend
 
-                    2. minimum additional fees(can be of any denom)
+                        If there is a recipient, this must not be zero. If there
+                        is not a recipient, this must be zero.
 
-                    3. optional recipient of fee based on
-                    `recipient_basis_points`
 
-                    4. if recipient is declared they will recieve the basis
-                    points of the fee (0-10,000)
+                        The recipient will receive additional_fee *
+                        recipient_basis_points / 10,000.
+
+                        The fee collector will receive the rest, i.e.
+                        additional_fee * (10,000 - recipient_basis_points) /
+                        10,000.
+                  description: >-
+                    MsgFee is the core of what gets stored on the blockchain to
+                    define a msg-based fee.
               pagination:
                 description: pagination defines an optional pagination for the request.
                 type: object
@@ -116675,39 +116682,49 @@ definitions:
     properties:
       msg_type_url:
         type: string
+        description: >-
+          msg_type_url is the type-url of the message with the added fee, e.g.
+          "/cosmos.bank.v1beta1.MsgSend".
       additional_fee:
-        title: >-
-          additional_fee can pay in any Coin( basically a Denom and Amount,
-          Amount can be zero)
+        description: >-
+          additional_fee is the extra fee that is required for the given message
+          type (can be in any denom).
         type: object
         properties:
           denom:
             type: string
           amount:
             type: string
-        description: |-
-          Coin defines a token with a denomination and an amount.
-
-          NOTE: The amount field is an Int which implements the custom method
-          signatures required by gogoproto.
       recipient:
         type: string
+        description: >-
+          recipient is an option address that will receive a portion of the
+          additional fee.
+
+          There can only be a recipient if the recipient_basis_points is not
+          zero.
       recipient_basis_points:
         type: integer
         format: int64
-    title: >-
-      MsgFee is the core of what gets stored on the blockchain
+        description: >-
+          recipient_basis_points is an optional portion of the additional fee to
+          be sent to the recipient.
+
+          Must be between 0 and 10,000 (inclusive).
 
-      it consists of four parts
 
-      1. the msg type url, i.e. /cosmos.bank.v1beta1.MsgSend
+          If there is a recipient, this must not be zero. If there is not a
+          recipient, this must be zero.
 
-      2. minimum additional fees(can be of any denom)
 
-      3. optional recipient of fee based on `recipient_basis_points`
+          The recipient will receive additional_fee * recipient_basis_points /
+          10,000.
 
-      4. if recipient is declared they will recieve the basis points of the fee
-      (0-10,000)
+          The fee collector will receive the rest, i.e. additional_fee * (10,000
+          - recipient_basis_points) / 10,000.
+    description: >-
+      MsgFee is the core of what gets stored on the blockchain to define a
+      msg-based fee.
   provenance.msgfees.v1.Params:
     type: object
     properties:
@@ -116750,42 +116767,49 @@ definitions:
           properties:
             msg_type_url:
               type: string
+              description: >-
+                msg_type_url is the type-url of the message with the added fee,
+                e.g. "/cosmos.bank.v1beta1.MsgSend".
             additional_fee:
-              title: >-
-                additional_fee can pay in any Coin( basically a Denom and
-                Amount, Amount can be zero)
+              description: >-
+                additional_fee is the extra fee that is required for the given
+                message type (can be in any denom).
               type: object
               properties:
                 denom:
                   type: string
                 amount:
                   type: string
-              description: >-
-                Coin defines a token with a denomination and an amount.
-
-
-                NOTE: The amount field is an Int which implements the custom
-                method
-
-                signatures required by gogoproto.
             recipient:
               type: string
+              description: >-
+                recipient is an option address that will receive a portion of
+                the additional fee.
+
+                There can only be a recipient if the recipient_basis_points is
+                not zero.
             recipient_basis_points:
               type: integer
               format: int64
-          title: >-
-            MsgFee is the core of what gets stored on the blockchain
+              description: >-
+                recipient_basis_points is an optional portion of the additional
+                fee to be sent to the recipient.
+
+                Must be between 0 and 10,000 (inclusive).
 
-            it consists of four parts
 
-            1. the msg type url, i.e. /cosmos.bank.v1beta1.MsgSend
+                If there is a recipient, this must not be zero. If there is not
+                a recipient, this must be zero.
 
-            2. minimum additional fees(can be of any denom)
 
-            3. optional recipient of fee based on `recipient_basis_points`
+                The recipient will receive additional_fee *
+                recipient_basis_points / 10,000.
 
-            4. if recipient is declared they will recieve the basis points of
-            the fee (0-10,000)
+                The fee collector will receive the rest, i.e. additional_fee *
+                (10,000 - recipient_basis_points) / 10,000.
+          description: >-
+            MsgFee is the core of what gets stored on the blockchain to define a
+            msg-based fee.
       pagination:
         description: pagination defines an optional pagination for the request.
         type: object

go.mod

@@ -44,7 +44,7 @@ require (
 	google.golang.org/genproto/googleapis/api v0.0.0-20240227224415-6ceb2ff114de
 	google.golang.org/grpc v1.63.2
 	google.golang.org/protobuf v1.34.1
-	gopkg.in/yaml.v2 v2.4.0
+	sigs.k8s.io/yaml v1.4.0
 )
 
 require (
@@ -201,11 +201,11 @@ require (
 	google.golang.org/genproto v0.0.0-20240227224415-6ceb2ff114de // indirect
 	google.golang.org/genproto/googleapis/rpc v0.0.0-20240401170217-c3f982113cda // indirect
 	gopkg.in/ini.v1 v1.67.0 // indirect
+	gopkg.in/yaml.v2 v2.4.0 // indirect
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 	gotest.tools/v3 v3.5.1 // indirect
 	nhooyr.io/websocket v1.8.10 // indirect
 	pgregory.net/rapid v1.1.0 // indirect
-	sigs.k8s.io/yaml v1.4.0 // indirect
 )
 
 replace (