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

Genesis Validation: Vesting #3418

Closed
jackzampolin opened this issue Jan 28, 2019 · 1 comment · Fixed by #3425
Closed

Genesis Validation: Vesting #3418

jackzampolin opened this issue Jan 28, 2019 · 1 comment · Fixed by #3425
Assignees
@alexanderbez
Labels
C:genesis relating to chain genesis
Milestone
v0.31.0 (Launch RC)

Comments

@jackzampolin
Copy link
Member

For each account we need to check that Vesting coins <= coins
@alexanderbez
Copy link
Contributor

alexanderbez commented Jan 29, 2019 •
edited
Loading

To be clear:

  • acc.OriginalVesting <= acc.Coins
  • acc.OriginalVesting all denoms must exist in acc.Coins
  • if any required vesting fields are provided, all must be provided

@alexanderbez alexanderbez self-assigned this Jan 29, 2019
@alexanderbez alexanderbez added this to the v0.31.0 (Launch RC) milestone Jan 29, 2019
@alexanderbez alexanderbez added the C:genesis relating to chain genesis label Jan 29, 2019
@alexanderbez alexanderbez mentioned this issue Jan 29, 2019
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexanderbez alexanderbez

Labels
C:genesis relating to chain genesis
Projects
None yet
Milestone
v0.31.0 (Launch RC)
Development

Successfully merging a pull request may close this issue.

R4R: Add Vesting Account Genesis Validation cosmos/cosmos-sdk
2 participants
@alexanderbez @jackzampolin