fix: Change call send amount to be an array of Coin #193
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The gnoclient API uses
MsgCall
where the send amount is an array ofCoin
. ACoin
is a struct with separate fields for Denom and Amount. However, the Gno Native Kit API uses a string such as "1000ugnot" (from the original version of gnoclient). Similar to PR #191 forMsgSend
, we want to track the current gnoclient API. This PR has three commits:MsgCall
to use an array ofCoin
for the Send amount. (The ProtobufCoin
type was already defined.) Runmake regenerate
to make the native interfaces. UpdategnoNativeService.convertCallRequest
to use theCoin
struct, similar to the previous change to convertSendRequest. Runnpm install @gnolang/gnonative
to install the new API on buf.build.call
andmakeCallTx
to take an array ofCoin
.BREAKING CHANGE: The only code which currently uses the Send amount in MsgCall is where gnokey-mobile calls "Register". After we merge this PR, we will update gnokey-mobile.