paritytech / substrate

Substrate: The platform for blockchain innovators
Apache License 2.0
8.39k stars 2.65k forks source link

let `initialize` function in `pallet-grandpa`/`pallet-babe` and other session managed pallets be `pub` #14556

Closed atenjin closed 1 year ago

atenjin commented 1 year ago

close #14555

In this pr, we hope to let those function be pub, which is called under OneSessionHandler::on_genesis_session function.

More details pleases refers to issue #14555

davxy commented 1 year ago

see https://github.com/paritytech/substrate/issues/14555#issuecomment-1643455714

KiChjang commented 1 year ago

Think we should close this; existing issue still has an unanswered comment, and it isn't clear yet that this PR is doing the right thing.