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.

Sign up for GitHub

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

Jump to bottom

adapt transfer e2e params tests to use gov v1 message #3705

Closed
1 task done
Tracked by #2010
crodriguezvega opened this issue May 31, 2023 · 1 comment
Closed
1 task done
Tracked by #2010

adapt transfer e2e params tests to use gov v1 message #3705

crodriguezvega opened this issue May 31, 2023 · 1 comment
Labels
20-transfer e2e
Milestone
v8.0.0

Comments

@crodriguezvega
Copy link
Contributor

Part of #2010

Pre-requisite:

Adjust TestSendEnabledParam and TestReceiveEnabledParam to use gov v1 message when the version of the chain supports it (keep using legacy params for versions that don't support gov v1).

Alternatively we could also not test legacy params anymore and test only with gov v1. Open to the opinion of others.
@crodriguezvega crodriguezvega mentioned this issue May 31, 2023
Closed
12 tasks
@crodriguezvega crodriguezvega added this to the v8.0.0 milestone May 31, 2023
@crodriguezvega crodriguezvega changed the title adapt transfer e2e params test to use gov v1 message adapt transfer e2e params tests to use gov v1 message May 31, 2023
@crodriguezvega
Copy link
Contributor Author

This was already taken care of in #3553!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
20-transfer e2e
Projects
None yet
Milestone
v8.0.0
Development

No branches or pull requests
1 participant
@crodriguezvega