Skip to content

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.

4 participants