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

Gentx is specific to staking #4969

Closed
tac0turtle opened this issue Aug 29, 2019 · 3 comments
Closed

Gentx is specific to staking #4969

tac0turtle opened this issue Aug 29, 2019 · 3 comments

Comments

@tac0turtle
Copy link
Member

The current gentx is specific to the staking module, for this reason, the case can be made to move the logic to live within the staking module.

@alexanderbez
Copy link
Contributor

thoughts here @rigelrozanski ?

@rigelrozanski
Copy link
Contributor

Genesis transactions is a general concept which should be abstracted beyond its current scope of staking. See this issue: #4771

gen-tx ought to remain distinguished for this reason, but once its abstracted, certain validity checks / logic would move back to staking from genutil

@tac0turtle
Copy link
Member Author

Closing this in favor of #4771

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants