LaconicNetwork/cosmjs-util
7
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Merge pull request #1510 from cosmos/pointless-toString

Browse Source 
Remove needless toString in MsgTransfer Amino converter
This commit is contained in:
Simon Warta 2023-11-15 17:53:07 +01:00 committed by GitHub
commit a8e151a617
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/stargate/src/modules/ibc/aminomessages.spec.ts
View File

@ -19,7 +19,7 @@ describe("AminoTypes", () => {
revisionNumber: BigInt("456"),
},
timeoutTimestamp: BigInt("789"),
memo: "",
memo: "something something",
};
const aminoTypes = new AminoTypes(createIbcAminoConverters());
const aminoMsg = aminoTypes.toAmino({
@ -39,7 +39,7 @@ describe("AminoTypes", () => {
revision_number: "456",
},
timeout_timestamp: "789",
memo: undefined,
memo: "something something",
},
};
expect(aminoMsg).toEqual(expected);

2
packages/stargate/src/modules/ibc/aminomessages.ts
View File

@ -71,7 +71,7 @@ export function createIbcAminoConverters(): AminoConverters {
}
: {},
timeout_timestamp: omitDefault(timeoutTimestamp)?.toString(),
memo: omitDefault(memo)?.toString(),
memo: omitDefault(memo),
}),
fromAmino: ({
source_port,