Skip to content

IBC-in skip support #2187

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

Open
grod220 opened this issue Oct 3, 2024 · 2 comments
Open

IBC-in skip support #2187

grod220 opened this issue Oct 3, 2024 · 2 comments
Labels
enhancement veil dex explorer related work

Comments

@grod220
Copy link
Contributor

grod220 commented Oct 3, 2024

Not all ibc'd tokens are fungible. For example:

Path A: TIA on Celestia -> Osmosis -> Penumbra
Path B: TIA on Celestia -> Penumbra

The tokens that eventually end up on Penumbra would not be the same. The canonical one is the one that came from the originating chain. So if users want to take their Osmosis TIA to Penumbra they must be able to "unwind":

TIA on Osmosis -> Celestia -> Penumbra

The Skip API is specifically designed for this purpose: https://api-docs.skip.money/docs/getting-started. It can assemble cosmos messages that allow for the automatic forwarding/unwinding through the native chain of each cosmos token.

Task: in the Dex explorer ibc/shielding page, utilize the api to support IBC'ing any token from any cosmos chain to Penumbra. Should be able to query Skip for route and pass the message to cosmos-kit. After that, there should not be any asset restriction on IBC'ing in.

@github-project-automation github-project-automation bot moved this to 🗄️ Backlog in Penumbra Web + Not Web Oct 4, 2024
@TalDerei TalDerei transferred this issue from penumbra-zone/dex-explorer Mar 28, 2025
@TalDerei TalDerei added the veil dex explorer related work label Mar 28, 2025
@TalDerei
Copy link
Contributor

TalDerei commented Apr 1, 2025

cc @vacekj as related context for in-flight IBC deposit work

@vacekj
Copy link
Member

vacekj commented Apr 14, 2025

If I understand this correctly, this is only an issue for the native deposit feature, which we don't plan on implementing in Veil. I assume the Skip widget handles this automatically for us (couldn't find anything in their docs to suggest otherwise).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement veil dex explorer related work
Projects
Status: 🗄️ Backlog
Development

No branches or pull requests

3 participants