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

Test params / Make gov proposals for all params #14358

Closed
3 tasks done
Tracked by #13456
julienrbrt opened this issue Dec 19, 2022 · 2 comments · Fixed by #14462
Closed
3 tasks done
Tracked by #13456

Test params / Make gov proposals for all params #14358

julienrbrt opened this issue Dec 19, 2022 · 2 comments · Fixed by #14462
Assignees

Comments

@julienrbrt
Copy link
Member

julienrbrt commented Dec 19, 2022

  • Create gov proposal for each parameter changes
  • Verify genesis of params changes at export of genesis
  • Verify parameters correctly set after import of a (migrated) genesis
@julienrbrt julienrbrt mentioned this issue Dec 19, 2022
54 tasks
@julienrbrt julienrbrt changed the title Make gov proposals for all params Test params / Make gov proposals for all params Dec 19, 2022
@julienrbrt julienrbrt self-assigned this Dec 19, 2022
@julienrbrt julienrbrt moved this from 📝 Todo to 💪 In Progress in Cosmos-SDK Dec 19, 2022
@julienrbrt julienrbrt moved this to 📝 Todo in Cosmos-SDK Dec 19, 2022
@likhita-809
Copy link
Contributor

is this issue completed @julienrbrt ?

@julienrbrt
Copy link
Member Author

is this issue completed @julienrbrt ?

I am about too. Taking longer due to the holiday 😬

@julienrbrt julienrbrt moved this from 💪 In Progress to 👀 Needs Review in Cosmos-SDK Jan 2, 2023
@github-project-automation github-project-automation bot moved this from 👀 Needs Review to 👏 Done in Cosmos-SDK Jan 2, 2023
@tac0turtle tac0turtle removed this from Cosmos-SDK May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants