Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add amino JSON support for MsgTransfer #8339

Closed
aaronc opened this issue Jan 14, 2021 · 1 comment
Closed

Add amino JSON support for MsgTransfer #8339

aaronc opened this issue Jan 14, 2021 · 1 comment
Milestone
v0.42.x

Comments

@aaronc
Copy link
Member

aaronc commented Jan 14, 2021

There are ongoing discussions on telegram about supporting IBC for the ledger app. I think the simplest way is to just add amino type registration back in for MsgTransfer. Am I correct in my understanding that this is currently the only end user-facing IBC transaction and that all other transactions would be sent by the relayer? If so, let's just support amino JSON for MsgTransfer - should be pretty quick and we can get it into v0.40.1.
@aaronc aaronc added this to the v0.40.1 milestone Jan 14, 2021
@amaury1093
Copy link
Contributor

duplicate of #8266. I propose to move the conversation there?

@amaury1093 amaury1093 mentioned this issue Jan 15, 2021
Closed
@aaronc aaronc removed the backlog label Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.42.x
Development

No branches or pull requests
2 participants
@aaronc @amaury1093