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.

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

Include support for ICS20-v2 #114

Open
2 of 4 tasks
srdtrk opened this issue Nov 20, 2024 · 1 comment
Open
2 of 4 tasks

Include support for ICS20-v2 #114

srdtrk opened this issue Nov 20, 2024 · 1 comment
Labels
20-transfer Issues related to the ICS20 application icebox Issues that will not be addressed for the time being type: feature Feature request

Comments

@srdtrk
Copy link
Member

srdtrk commented Nov 20, 2024

Summary

Support the latest version of the transfer application

Problem Definition

Support the latest ICS20 specs. This issue needs discussion until the following are resolved:

  • What are the new gas costs?
  • How would solvers work with multi-denom packets?
  • Will we have transfer+action workflow supported?

Use cases

  • Multi denom transfers
  • Multi hop transfers

Proposal

In our initial implementation we don't need to include support for multihop transfers where EVM is a middle-chain for simplicity.


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged/assigned
  • Estimate provided
@srdtrk srdtrk added 20-transfer Issues related to the ICS20 application icebox Issues that will not be addressed for the time being type: feature Feature request labels Nov 20, 2024
@womensrights
Copy link

As we discussed internally:

  • if we support two versions of ics-20 at launch, this will add additional complexity
  • there are benefits to ics-20v2 related to the denom trace
  • we will need a forwarding solution for eureka

Therefore, we should use ics20v2 over ics20 v1

@gjermundgaraba gjermundgaraba self-assigned this Jan 3, 2025
@gjermundgaraba gjermundgaraba removed the icebox Issues that will not be addressed for the time being label Jan 3, 2025
@gjermundgaraba gjermundgaraba moved this from Backlog to In progress in IBC-GO Eureka Jan 22, 2025
@gjermundgaraba gjermundgaraba linked a pull request Jan 28, 2025 that will close this issue
6 tasks
@gjermundgaraba gjermundgaraba moved this from In progress to In review in IBC-GO Eureka Feb 1, 2025
@srdtrk srdtrk removed the status in IBC-GO Eureka Feb 4, 2025
@srdtrk srdtrk added the icebox Issues that will not be addressed for the time being label Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
20-transfer Issues related to the ICS20 application icebox Issues that will not be addressed for the time being type: feature Feature request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants