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

Check in transfer e2e test that all balances response returns human-readable denomination #3768

Closed
2 tasks done
crodriguezvega opened this issue Jun 6, 2023 · 0 comments · Fixed by #4642
Closed
2 tasks done
Assignees
Milestone

Comments

@crodriguezvega
Copy link
Contributor

crodriguezvega commented Jun 6, 2023

Pre-requisite:

Add a step in TestMsgTransfer_Succeeds_Nonincentivized after tokens have been transferred to chain B and query all balances of chainBAddress using the resolve_denom flag to check that the balance of the IBC voucher is returned with a human-readable denomination (instead of the IBC hash).

Adding this tentatively to v8.0.0 milestone and assuming that upgrade to SDK Eden will also be part of the same release.

@crodriguezvega crodriguezvega added this to the v8.0.0 milestone Jun 6, 2023
@crodriguezvega crodriguezvega moved this to Todo in ibc-go Jun 6, 2023
@crodriguezvega crodriguezvega moved this from Todo to Backlog in ibc-go Jun 6, 2023
@crodriguezvega crodriguezvega changed the title Add check in transfer e2e test Check in transfer e2e test that all balances response returns human-readable denomination Jun 6, 2023
@crodriguezvega crodriguezvega moved this from Backlog to Todo in ibc-go Aug 17, 2023
@crodriguezvega crodriguezvega moved this from Todo to On hold in ibc-go Sep 3, 2023
@crodriguezvega crodriguezvega self-assigned this Sep 11, 2023
@crodriguezvega crodriguezvega moved this from On hold to In progress in ibc-go Sep 12, 2023
@crodriguezvega crodriguezvega linked a pull request Sep 12, 2023 that will close this issue
9 tasks
@github-project-automation github-project-automation bot moved this from In progress to Done in ibc-go Sep 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant