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

Document Instant Redelegation Safety #1670

Closed
rigelrozanski opened this issue Jul 13, 2018 · 6 comments
Closed

Document Instant Redelegation Safety #1670

rigelrozanski opened this issue Jul 13, 2018 · 6 comments
Assignees
Labels
C:x/staking T:Docs Changes and features related to documentation.

Comments

@rigelrozanski
Copy link
Contributor

Document instant re-delegation and explain why it's safe, tabulate all possible slashing scenarios with redelegation

(CC @sunnya97)

@rigelrozanski rigelrozanski added T:Docs Changes and features related to documentation. C:x/staking prelaunch labels Jul 13, 2018
@gamarin2
Copy link
Contributor

Do we really need this prelaunch?

@ebuchman
Copy link
Member

Do we really need this prelaunch?

Part of the larger review of the staking module. I think we should defntly try to document/justify as much as possible everything in this module given the complexity and the issues we've had so far with the edge cases ...

@gamarin2
Copy link
Contributor

@ebuchman I wasn't aware the feature itself was in the module already :p I was talking about that, not about the documentation.

@rigelrozanski
Copy link
Contributor Author

Yeah instant redelegation is a prerequisite to a healthy DPOS system - i'm confident it's safe based on numerous conversations internally, just need to clearly explicitly lay it out in the docs. Yes, it's already built (actually not too complex in implementation) the justification is a bit more complicated

@jackzampolin
Copy link
Member

Close in favor of the tracking tracking issue.

@cjcobb23
Copy link

Can anyone point me to where this doc is?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/staking T:Docs Changes and features related to documentation.
Projects
None yet
Development

No branches or pull requests

7 participants