-
Notifications
You must be signed in to change notification settings - Fork 144
Closed
Labels
C-simpleMinor changes changes, no additional research needed. Good first issue/review.Minor changes changes, no additional research needed. Good first issue/review.S-babeissues related to block production functionality.issues related to block production functionality.
Description
Describe the bug
- Currently, the
VerificationManager.VerifyBlock
method changes the epoch state if the header passed as argument is 1 - It does not make sense for the verification to change the epoch state
- Makes more sense to set the first slot at finalization as said here fix(lib/babe): fix setting first slot of network, fix loading BABE epoch params #1640
Metadata
Metadata
Assignees
Labels
C-simpleMinor changes changes, no additional research needed. Good first issue/review.Minor changes changes, no additional research needed. Good first issue/review.S-babeissues related to block production functionality.issues related to block production functionality.
Type
Projects
Status
Done