-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Untie staking from genesis #4422
Comments
As the code is already structured you should be able to get rid of the staking module entirely when dealing with app genesis. however you should simply not be importing any |
Hmmm, how do you plan on providing the validator set to tendermint? Just statically defined/POA style? |
Yeah the expected types for this decoupled genutil will need to have certain key information pertaining to pubkey/power which will input into tendermint |
I believe most parts of this issue was resolved with the module gen refactor. I can tackle the rest once #4757 is merged |
@fedekunze I've broken out part of this issue to #4771 which should be taken on independantly (and isn't a particularly easy task I don't think) |
@rigelrozanski I was thinking of using expected interfaces/dummies for the staking MsgCreateValidator instead. But as your approach is more general I guess we can then close this issue |
roger |
Summary
Currently the staking module seems vital to boot any dapp built on top of the SDK
Problem Definition
Would be nice to remove this necessity, have the genesis and all the basic features working without specific modules, or modules that not every app should use.
@fedekunze @rigelrozanski @alexanderbez @jackzampolin
For Admin Use
The text was updated successfully, but these errors were encountered: