Skip to content

[WIP] Remove staking logic and data from the host#395

Closed
HoOngEe wants to merge 6 commits intoCodeChain-io:moldfrom
HoOngEe:remove/staking-in-host
Closed

[WIP] Remove staking logic and data from the host#395
HoOngEe wants to merge 6 commits intoCodeChain-io:moldfrom
HoOngEe:remove/staking-in-host

Conversation

@HoOngEe
Copy link
Contributor

@HoOngEe HoOngEe commented May 18, 2020

The commits should be modified following the ordering rules defined in #388.
It can be merged only after #335

HoOngEe added 4 commits May 18, 2020 17:41
It removes CurrentValidatorSet and NextValidatorSet which is the
new types adding conversions to CompactValidators from CurrentValidators
and NextValidators, and vice versa.
The names CompactValidators and CompactValidatorEntry are originated
from the host side Validator type. However, now host don't have to
store staking-detailed information and only stores compact information.
@HoOngEe HoOngEe assigned HoOngEe and unassigned HoOngEe May 18, 2020
@HoOngEe HoOngEe added do-not-merge Do not merge this PR mold Module system labels May 18, 2020
HoOngEe added 2 commits May 18, 2020 18:04
The host no more stores specific staking information like delegation.
It should be renamed to more abstract one such as voting_power.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge Do not merge this PR mold Module system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants