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

Enable parachains only if synchronized #1566

Merged
merged 7 commits into from
Apr 21, 2023
Merged

Conversation

iceseer
Copy link
Contributor

@iceseer iceseer commented Apr 12, 2023

Description of the Change

Enable parachains only if Babe::State::Synchronized. Otherwise disabled.

iceseer added 2 commits April 12, 2023 13:42
Signed-off-by: iceseer <iceseer@gmail.com>
Signed-off-by: iceseer <iceseer@gmail.com>
turuslan
turuslan previously approved these changes Apr 12, 2023
@iceseer iceseer requested a review from kamilsa April 17, 2023 19:19
turuslan
turuslan previously approved these changes Apr 21, 2023
Signed-off-by: iceseer <iceseer@gmail.com>
@iceseer iceseer enabled auto-merge (squash) April 21, 2023 10:22
@iceseer iceseer merged commit ee75e51 into master Apr 21, 2023
@iceseer iceseer deleted the feature/enable_parachains branch April 21, 2023 11:22
iceseer added a commit that referenced this pull request May 9, 2023
This reverts commit ee75e51.

Signed-off-by: iceseer <iceseer@gmail.com>

# Conflicts:
#	core/consensus/babe/impl/babe_impl.cpp
iceseer added a commit that referenced this pull request May 9, 2023
This reverts commit ee75e51.

Signed-off-by: iceseer <iceseer@gmail.com>

# Conflicts:
#	core/consensus/babe/impl/babe_impl.cpp
iceseer added a commit that referenced this pull request May 11, 2023
* Revert "Enable parachains only if synchronized (#1566)"

This reverts commit ee75e51.

Signed-off-by: iceseer <iceseer@gmail.com>

# Conflicts:
#	core/consensus/babe/impl/babe_impl.cpp

* disable protocol handling if turned-off

Signed-off-by: iceseer <iceseer@gmail.com>

---------

Signed-off-by: iceseer <iceseer@gmail.com>
Co-authored-by: kamilsa <kamilsa16@gmail.com>
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.

3 participants