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

Add "extra data" field to TokenTransfer*Context::escrow_coins_*() methods #839

Closed
Tracked by #554 ...
plafer opened this issue Aug 22, 2023 · 0 comments · Fixed by #836
Closed
Tracked by #554 ...

Add "extra data" field to TokenTransfer*Context::escrow_coins_*() methods #839

plafer opened this issue Aug 22, 2023 · 0 comments · Fixed by #836
Labels
A: breaking Admin: breaking change that may impact operators
Milestone

Comments

@plafer
Copy link
Contributor

plafer commented Aug 22, 2023

In the Sovereign SDK, we need to pass the token address as extra information when escrowing tokens. We can generalize this to allows implementors to pass in arbitrary data to the escrow_coins_*() methods.

@plafer plafer added the A: breaking Admin: breaking change that may impact operators label Aug 22, 2023
@plafer plafer self-assigned this Aug 22, 2023
@github-project-automation github-project-automation bot moved this to 📥 To Do in ibc-rs Aug 22, 2023
@Farhad-Shabani Farhad-Shabani moved this from 📥 To Do to 🏗️ In Progress in ibc-rs Aug 22, 2023
@Farhad-Shabani Farhad-Shabani added this to the v0.49.0 milestone Dec 8, 2023
@github-project-automation github-project-automation bot moved this from 🏗️ In Progress to ✅ Done in ibc-rs Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: breaking Admin: breaking change that may impact operators
Projects
None yet
2 participants