Skip to content

fix: Reinstate editor singleton registry#26

Merged
dcalhoun merged 2 commits intotrunkfrom
fix/reinstate-editor-singleton-registry
Sep 23, 2024
Merged

fix: Reinstate editor singleton registry#26
dcalhoun merged 2 commits intotrunkfrom
fix/reinstate-editor-singleton-registry

Conversation

@dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Sep 20, 2024

Description

Reinstate disabling the editor sub-registry so that a singleton registry is
used. This allows third-party blocks to query the block editor state.

Additionally, remove what appears to be a redundant invocation of
getEditorSettings that may have been erroneously left in place during
#24. It appears the EditorProvider already invokes getEditorSettings.

Testing Instructions

Verify the editor — local or remote — launches without errors.

Verify Jetpack AI queries that require block content succeed; e.g., requesting
Jetpack AI modify an existing block.

This appears to already be invoked by the `EditorProvider`. The
redundant invocation was likely erroneously left in place when we began
using the `EditorProvider` recently.

#24
https://github.com/WordPress/gutenberg/blob/219065c3be5881a68a8667fda9c04c7b7f546763/packages/editor/src/components/provider/index.js#L178
Reinstate the editor singleton registry so that third-party blocks may
query the block editor state.
@dcalhoun dcalhoun added the [Type] Bug An existing feature does not function as intended label Sep 20, 2024
@dcalhoun dcalhoun marked this pull request as ready for review September 20, 2024 20:46
@dcalhoun dcalhoun requested a review from geriux September 20, 2024 20:46
@dcalhoun dcalhoun merged commit 429b3b1 into trunk Sep 23, 2024
@dcalhoun dcalhoun deleted the fix/reinstate-editor-singleton-registry branch September 23, 2024 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Bug An existing feature does not function as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants