Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: Add IBC TransferV2 #611

Open
wants to merge 4 commits into
base: feat/ibc_eureka
Choose a base branch
from

Conversation

kulikthebird
Copy link
Contributor

No description provided.

@kulikthebird kulikthebird force-pushed the tkulik/feat/ibc_transfer_v2 branch from ddaf289 to 0322ca5 Compare January 27, 2025 09:29
@kulikthebird kulikthebird force-pushed the tkulik/feat/ibc_transfer_v2 branch from 0322ca5 to 2a1a4ad Compare January 27, 2025 13:18
@kulikthebird kulikthebird marked this pull request as ready for review January 27, 2025 14:46
@kulikthebird kulikthebird force-pushed the tkulik/feat/ibc_transfer_v2 branch from 3c641d9 to eec5434 Compare January 30, 2025 08:16
@kulikthebird kulikthebird force-pushed the tkulik/feat/ibc_transfer_v2 branch from eec5434 to 2e4deab Compare January 30, 2025 11:42
Copy link
Collaborator

@chipshort chipshort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice and simple. This of course also need to wait for the cosmwasm release.

types/msg.go Outdated
//
// There is no difference between setting this to `None` or an empty string.
//
// This field is only supported on chains with CosmWasm >= 2.0 and silently ignored on older chains.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Leftover from copying? Chains older than 3.0 won't support transfer v2 at all.

@kulikthebird kulikthebird changed the base branch from main to feat/ibc_eureka February 3, 2025 13:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants