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

User should not be able to both slash and stake an argument #751

Open
2 tasks
truted opened this issue Jul 17, 2019 · 2 comments
Open
2 tasks

User should not be able to both slash and stake an argument #751

truted opened this issue Jul 17, 2019 · 2 comments

Comments

@truted
Copy link
Contributor

truted commented Jul 17, 2019

  • reject Staking msg if argument is already slashed
  • reject Slashing msg if argument is already staked
@shanev
Copy link
Contributor

shanev commented Jul 19, 2019

We are handling this on the client side, but will be good to get this in at some point.

@shanev
Copy link
Contributor

shanev commented Jul 19, 2019

To make this work, we have to generalize the slashing module (add expected keepers, interfaces, exported types and aliases). Then staking can depend on slashing, and slashing can depend on staking without cyclic dependencies.

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

No branches or pull requests

2 participants