Merge pull request #382 from crypto-matto/fix/MsgDeposit

Problem: Fix MsgDeposit proposa_id field
crypto-org-chain · Jul 25, 2022 · dcf16f6 · dcf16f6
2 parents 63d168c + 478252b
commit dcf16f6
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,9 @@
 
 All notable changes to this project will be documented in this file.
 
+## v1.0.7
+- Fix incorrect `proposal_id` type of `MsgDeposit` in Amino format
+
 ## v1.0.6
 - Added support for `MsgDeposit` and `MsgSubmitProposal` in Amino format
 

diff --git a/lib/src/transaction/msg/v2/gov/v2.MsgDeposit.spec.ts b/lib/src/transaction/msg/v2/gov/v2.MsgDeposit.spec.ts
@@ -120,7 +120,7 @@ describe('Testing MsgDeposit', function () {
             const rawMsg: legacyAmino.Msg = {
                 type: 'cosmos-sdk/MsgDeposit',
                 value: {
-                    proposal_id: Long.fromNumber(1244000, true),
+                    proposal_id: '1244000',
                     depositor: 'tcro184lta2lsyu47vwyp2e8zmtca3k5yq85p6c4vp3',
                     amount: [coin.toCosmosCoin()],
                 },

diff --git a/lib/src/transaction/msg/v2/gov/v2.MsgDeposit.ts b/lib/src/transaction/msg/v2/gov/v2.MsgDeposit.ts
@@ -38,7 +38,7 @@ export const msgDepositV2 = function (config: InitConfigurations) {
 
         // eslint-disable-next-line class-methods-use-this
         toRawAminoMsg(): legacyAmino.Msg {
-            const proposal = Long.fromNumber(this.proposalId.toNumber(), true);
+            const proposal = this.proposalId.toString();
             return {
                 type: 'cosmos-sdk/MsgDeposit',
                 value: {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@crypto-org-chain/chain-jslib",
-  "version": "1.0.6",
+  "version": "1.0.7",
   "description": "Crypto.org Chain JavaScript library",
   "author": "Crypto.org <[email protected]>",
   "license": "Apache-2.0",