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 experimental full site editing blocks not being registered #39415

Merged
merged 1 commit into from
Mar 14, 2022

Conversation

talldan
Copy link
Contributor

@talldan talldan commented Mar 14, 2022

What?

Fixes a an issue introduced in #39180 - experimental full site editing blocks were not being registered.

Props to @michalczaplinski for noticing this and finding the PR that introduced the bug.

Why?

In #39180, some block settings were moved to a new file the wordpress-5.9 folder.

The issue is that this file wasn't being loaded, so the editor settings (which included __unstableEnableFullSiteEditingBlocks) were not being applied.

How?

Added a new line to the load.php to require the file.

Testing Instructions

  1. Open the post editor
  2. Try adding a comments-query-loop block

Expected: The block should be available and can be added.

@talldan talldan added [Type] Bug An existing feature does not function as intended [Feature] Blocks Overall functionality of blocks labels Mar 14, 2022
@talldan talldan requested a review from TimothyBJacobs as a code owner March 14, 2022 02:39
@talldan talldan self-assigned this Mar 14, 2022
@talldan talldan requested a review from spacedmonkey as a code owner March 14, 2022 02:39
Copy link
Contributor

@michalczaplinski michalczaplinski left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thank you @talldan ! 🙏

@ockham
Copy link
Contributor

ockham commented Mar 15, 2022

Should we add some (basic) e2e test coverage for the new (experimental) Comment blocks?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feature] Blocks Overall functionality of blocks [Type] Bug An existing feature does not function as intended
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants