-
Notifications
You must be signed in to change notification settings - Fork 2.7k
AURA: Switch to CurrentSlot
instead of LastTimestamp
#8023
Conversation
This switches AURA to use `CurrentSlot` instead of `LastTimestamp`.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As discussed on Element we should be able to remove the slot number inherent data (we are reading it from predigest anyway).
Added |
Co-authored-by: André Silva <123550+andresilva@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/paritytech/substrate/blob/master/primitives/consensus/aura/src/inherents.rs#L50
Do you mind adding a comment here saying that this can be removed in the future (maybe with a link to this PR)?
Runtimes need to remove the inherent for the module. Missing this one: https://github.com/paritytech/substrate/blob/master/frame/aura/src/mock.rs#L43 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM,
I'm also happy to see those ProvideInherent implementation which just check some other pallet calls removed, the logic is more simple now, IMO.
This switches AURA to use
CurrentSlot
instead ofLastTimestamp
.Based on: #8020 (so only the last commit is important at the moment)