Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Replace Block Height to Block Hash map #68

Closed
fedekunze opened this issue Jun 4, 2021 · 0 comments · Fixed by #80
Closed

Replace Block Height to Block Hash map #68

fedekunze opened this issue Jun 4, 2021 · 0 comments · Fixed by #80
Assignees

Comments

@fedekunze
Copy link
Contributor

The HeaderHash getter and setter mapping from the store can be replaced by the staking module Keeper's HistoricalInfo that contains the header and the validator set.

Actionable Items:

  • Add staking keeper
  • Create StakingKeeper interface on the interfaces.go file, defining the expected functions:
GetHistoricalInfo(ctx sdk.Context, height int64) (types.HistoricalInfo, bool)
  • Remove the header hash store functions, keys, etc
@fedekunze fedekunze added the good first issue Good for newcomers label Jun 4, 2021
@fedekunze fedekunze self-assigned this Jun 7, 2021
@fedekunze fedekunze removed the good first issue Good for newcomers label Jun 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant