Skip to content

Conversation

@kianenigma
Copy link
Member

@kianenigma kianenigma commented Jul 11, 2023

closes #620 and in general preps substrate module for pba3.

Copy link
Member

@wirednkod wirednkod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some nips and tucks only (and correction of colors)

Comment on lines +6 to +9
/// Use manual seal, which basically produces blocks at fixed intervals, at the given interval.
///
/// Example: `manual-seal-3000` will produce a block every 3 seconds.
ManualSeal(u64),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think, "manual seal" usually refers to when you wire this thing up to the RPC and users manually author blocks by sending rpc commands to do so. Calling this interval seal might make more sense.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good point, it is using manual-seal under the hood, but I have indeed created my own background task to manually trigger it every x milliseconds.

@kianenigma
Copy link
Member Author

I seem to have messed up and causes some unneeded conflicts, will start from scratch in a fresh branch.

@kianenigma kianenigma closed this Jul 19, 2023
@kianenigma kianenigma deleted the kiz-substrate-pba3 branch July 19, 2023 10:46
@nuke-web3
Copy link

nuke-web3 commented Jul 19, 2023

@kianenigma not you - just the folder go renamed to account for module reordering - should be OK to reopen - just make tou working dir syllabus/5-Substrate now and assets/img/5-Substrate

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.

Tracking Issue Substrate Module PBA3

5 participants