Don't use GetParams #363
Merged
Don't use GetParams #363
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
fedekunze
approved these changes
Mar 6, 2022
fedekunze
added a commit
that referenced
this pull request
Mar 7, 2022
* release: bump Evmos version to v2 (#351) * fix changelog * fix * swagger * swagger * fix * go mod * app: upgrade handler for v2 (#352) * app: set up upgrade handler for v2 * use newer sdk version instructions * run migrations and set modules versions to 1 * add TODO * use osmosis style upgrade handlers * remove upgrade handler for olympus mons * make note of inflation module change as well * add store loaders upgrade * set the params for erc20 module * set the params for claims * pass in keeers to upgrade handler * apply upgrade at specified height * remove store loaders upgrade * bump the claims consensus version * add binaries info to upgrade info * add new claims params * add comment * insert sdk fork and upgrade handler * update dependencies to v2 * remove old fork * docs: use `go install` for cosmovisor (#356) * docs: add `v1.0.0` to cosmovisor install (#357) * remove old evmos * go mod tidy * fix upgrade handler to not replace version map * docs: cosmovisor v2 upgrade preparation docs (#358) * docs: fix rpc urls * docs: v2 upgrade instructions for cosmovisor * add details for latest v1 release * release: `v1.1.2` changelog (#359) * release: v1.1.0 (#324) * release: v1.1.1 (#346) * fix conflicts * conflicts Co-authored-by: Prajjwol Gautam <not@acyb.org> * app: upgrade v1.1.2 logic to v2 (#354) * app: set up upgrade handler for v2 use newer sdk version instructions run migrations and set modules versions to 1 add TODO use osmosis style upgrade handlers remove upgrade handler for olympus mons make note of inflation module change as well add store loaders upgrade set the params for erc20 module set the params for claims pass in keeers to upgrade handler apply upgrade at specified height remove store loaders upgrade bump the claims consensus version add binaries info to upgrade info add new claims params add comment throw upgrade plan at cosmovisor clean up changes that should be in v2 clean up changes that should be in v2, again remove extra dependency * go mod tidy * add changelog * Update fork code to use something closer to upgrade module * Fix improt error * remove upgrade height check * log errors * panic on error instead * add back height gate * add missing bracket * erc20 migrator * nolint * claims migration * add TODO * rm erc20 upgrade * don't register migration for claims * update changelog Co-authored-by: ValarDragon <dojha12@gmail.com> Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com> * update fork.go Co-authored-by: Prajjwol Gautam <not@acyb.org> Co-authored-by: Akash Khosla <8366960+khoslaventures@users.noreply.github.com> Co-authored-by: ValarDragon <dojha12@gmail.com> * upgrade handler * cleanup * register erc20 migration * deps: bump ethermint to v0.11.0 (#360) * deps: bump ethermint to v0.11.0 * changelog * bump ethermint Co-authored-by: Federico Kunze Küllmer <31522760+fedekunze@users.noreply.github.com> Co-authored-by: Prajjwol Gautam <not@acyb.org> Co-authored-by: ValarDragon <dojha12@gmail.com> Co-authored-by: Federico Kunze Küllmer <federico.kunze94@gmail.com> * Merge pull request from GHSA-5jgq-x857-p8xw * fix: only enable claims from authorized channels * changelog * changelog 2 * fix tests * rename to authorized * update forks * upgrade height * check for EVM chains * bonded ration adjustment * update genesis * fixes * swagger * fix: don't use GetParams (#363) Co-authored-by: Akash Khosla <8366960+khoslaventures@users.noreply.github.com> Co-authored-by: Prajjwol Gautam <not@acyb.org> Co-authored-by: ValarDragon <dojha12@gmail.com> Co-authored-by: Dev Ojha <ValarDragon@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Closes: #XXX
Fixes UnmarshalJSON error when getting params from the store
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
PR review checkboxes:
I have...
Unreleasedsection inCHANGELOG.mdCode maintenance:
I have...
godocand code comments.docs/) or specification (x/<module>/spec/)Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add your handle next to the items reviewed if you only reviewed selected items.
I have...