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

go/staking: Remove Approve/Withdraw support #2043

Merged
merged 1 commit into from Aug 28, 2019

Conversation

Yawning
Copy link
Contributor

@Yawning Yawning commented Aug 28, 2019

The staking ledger backend doesn't need to be ERC20 compliant to provide
staking services, and this is a dramatic reduction in code.

Fixes #2021

@Yawning Yawning added p:1 Priority: core feature c:registry Category: entity/node/runtime registry service golang c:consensus/tendermint Category: Tendermint-based consensus labels Aug 28, 2019
@Yawning Yawning requested a review from kostko August 28, 2019 10:57
@Yawning Yawning self-assigned this Aug 28, 2019
The staking ledger backend doesn't need to be ERC20 compliant to provide
staking services, and this is a dramatic reduction in code.
@Yawning Yawning force-pushed the yawning/feature/stake-simplification branch from 7d86b91 to c775fb3 Compare August 28, 2019 13:44
@Yawning Yawning merged commit 6e779a2 into master Aug 28, 2019
@Yawning Yawning deleted the yawning/feature/stake-simplification branch August 28, 2019 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:consensus/tendermint Category: Tendermint-based consensus c:registry Category: entity/node/runtime registry service golang p:1 Priority: core feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

go/staking: Remove the Approve, Withdraw and related calls
2 participants