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

Fixes #77 (NewConsensusClient function) #78

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

18aaddy
Copy link

@18aaddy 18aaddy commented Oct 24, 2024

As described in the issue (#77 ), I have changed the following:

  • BlockRecv and FinalizedBlockRecv types to chan *common.Block
  • The db variable to use ConfigDB right now. It can be changed to FileDB as well depending on the need
  • Correct handling of config.Checkpoint according to Helios codebase
  • Also had to change the BlockSend field of Inner struct from chan common.Block to chan *common.Block to adjust for changes and Also the New() function of Inner struct accordingly.

More detailed description are also present as comments in the code

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

Successfully merging this pull request may close these issues.

1 participant