-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Renames parent selection boolean param and improves docs #48677
Conversation
Co-authored-by: Nik Tsekouras <16275880+ntsekouras@users.noreply.github.com> Co-authored-by: George Mamadashvili <240569+Mamaduka@users.noreply.github.com>
Size Change: -2 B (0%) Total Size: 1.34 MB
ℹ️ View Unchanged
|
Flaky tests detected in b6ccfa0. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4313472016
|
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.
Thank you, @draganescu!
The fix works as before. Tested using instructions from #48204.
* Renames parent selection boolean param and improves docs Co-authored-by: Nik Tsekouras <16275880+ntsekouras@users.noreply.github.com> Co-authored-by: George Mamadashvili <240569+Mamaduka@users.noreply.github.com> * better wording for action docs --------- Co-authored-by: Nik Tsekouras <16275880+ntsekouras@users.noreply.github.com> Co-authored-by: George Mamadashvili <240569+Mamaduka@users.noreply.github.com>
I just cherry-picked this PR to the wp/6.2 branch to get it included in the next release: d99c758 |
This is a small code quality and documentation PR to improve how #48204 gets merged to core.
Co-authored-by: Nik Tsekouras 16275880+ntsekouras@users.noreply.github.com
Co-authored-by: George Mamadashvili 240569+Mamaduka@users.noreply.github.com
What?
Rename the param to optionally select parent block of the
selectPreviousBlock
action tofallbackToParent
Make docs more explicit.
Remove one useless variable.
All tests should pass.