-
Notifications
You must be signed in to change notification settings - Fork 17
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
v2.2.0 upgrade #129
v2.2.0 upgrade #129
Conversation
* feat: add upgrade handler * chore: fix bech32 error * chore: fix typo * feat: add ante handler to block account * chore: fix vesting unlock time * Update app/upgrades/v1/upgrade.go Co-authored-by: Anil K <[email protected]> * Update app/upgrades/v1/upgrade.go Co-authored-by: Anil K <[email protected]> * chore: add depgaurd config * chore: update linter * chore: try fixing linter --------- Co-authored-by: Anil K <[email protected]>
* Add Wasm to Snapshot * Update app/app.go --------- Co-authored-by: Anil K <[email protected]>
Co-authored-by: Anil K <[email protected]>
Co-authored-by: Anil K <[email protected]>
* chore: add upgrade handler `v2.2.0` * fix claim records * update time * fix panic * changes * add claim records from snapshots * suggestions * review changes * review changes * Update app/upgrades/v2.2.0/upgrade.go * fix amount sim bug * add logs * log * log * Fix airdrop amounts * fix build * fix * review changes * Update x/claim/keeper/claim.go * test calc * small fix * another handle * handle more coin errors * revert * add snapshot1 data directly * update data * sub original claim allocation * review changes * Update app/upgrades/v2.2.0/upgrade.go * update go.mod --------- Co-authored-by: Anil K <[email protected]> Co-authored-by: Anil K <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## releases/v2.x #129 +/- ##
================================================
Coverage ? 54.74%
================================================
Files ? 25
Lines ? 2285
Branches ? 0
================================================
Hits ? 1251
Misses ? 959
Partials ? 75 |
Description
Closes: #XXXX
Author Checklist
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.
I have...
!
to the type prefix if API or client breaking changeCHANGELOG.md
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...
!
in the type prefix if API or client breaking change