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

Update Transfer Authz implementation to account for multi denom #5802

Closed
Tracked by #5793
chatton opened this issue Feb 1, 2024 · 2 comments · Fixed by #6252
Closed
Tracked by #5793

Update Transfer Authz implementation to account for multi denom #5802

chatton opened this issue Feb 1, 2024 · 2 comments · Fixed by #6252
Assignees
@charleenfei
Labels
20-transfer
Milestone
ICS20 v2 (multi d...

Comments

@chatton
Copy link
Contributor

chatton commented Feb 1, 2024 •
edited
Loading

The MsgTransfer authz implementation needs to be updated to support multi denom.

See the PoC implementation
@chatton chatton mentioned this issue Feb 1, 2024
Closed
24 tasks
@crodriguezvega crodriguezvega added this to the ICS20 v2 milestone Feb 1, 2024
@chatton
Copy link
Contributor Author

chatton commented Apr 8, 2024

Note: I believe migrations shouldn't be required for this, but we should make sure to confirm this as part of this issue.

@charleenfei charleenfei self-assigned this May 2, 2024
@charleenfei charleenfei changed the title [ICS20-2] Update Transfer Authz implementation to account for multi denom. Update Transfer Authz implementation to account for multi denom May 2, 2024
@charleenfei charleenfei mentioned this issue May 2, 2024
Merged
10 tasks
@crodriguezvega
Copy link
Contributor

Closed by #6252.

@coderabbitai coderabbitai bot mentioned this issue Jun 4, 2024
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@charleenfei charleenfei

Labels
20-transfer
Projects
ibc-go
Archived in project
Milestone
ICS20 v2 (multi denom) alpha
Development

Successfully merging a pull request may close this issue.

imp: update transfer authz implementation to account for multi denom transfers cosmos/ibc-go
3 participants
@crodriguezvega @chatton @charleenfei