-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add upgrade v3 to update endtime for validator vesting accounts #439
Conversation
Codecov Report
@@ Coverage Diff @@
## main #439 +/- ##
=======================================
Coverage 56.88% 56.88%
=======================================
Files 74 74
Lines 5805 5806 +1
=======================================
+ Hits 3302 3303 +1
Misses 2181 2181
Partials 322 322
|
app/upgrades/v3/upgrades.go
Outdated
return nil, err | ||
} | ||
ak.GetAccount(ctx, accAddr) | ||
// TODO set endtime for vesting account |
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.
You would need to cast to *vestingtypes.ContinuousVestingAccount
to access the field, apply some safety checks and then update the data via ak.SetAccount
newEndTime int64 = 1688220000 | ||
) | ||
|
||
var addresses = []string{ |
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.
These accounts do not exists on mainnet and will break the migration:
tgrade1ukqqgpftlmwf2ylkvfj467e6wynef05lqcrtjn
tgrade19g64psls2eqwd34nu03c2pcc45jptzxpmwmzu9
f8b7894
to
dc367e1
Compare
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
* Add upgrade v3 to update endtime for validator vesting accounts * Add addresses * Add logic for updating end time for vesting account * Migration test example (#454) * Assert end time to be update correctly Co-authored-by: Alexander Peters <alpe@users.noreply.github.com>
Resolves #420