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

fix: infinite loop caused within block component #78

Merged
merged 1 commit into from
Feb 20, 2021

Conversation

jmikrut
Copy link
Member

@jmikrut jmikrut commented Feb 20, 2021

Description

Fixes #77 - wow. We were setting default values for the labels prop within the Blocks component itself, which would cause the memoizedValidate callback to be recreated every render. Voila, infinite loop.

  • I have read and understand the CONTRIBUTING.md document in this repository

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • Existing test suite passes locally with my changes

@jmikrut jmikrut merged commit d343add into master Feb 20, 2021
@denolfe denolfe deleted the fix/block-conditional branch July 7, 2022 02:37
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.

Adding conditional to blocks field type causes admin UI to hang
1 participant