Conversation
Pull Request Test Coverage Report for Build 3a5c5aca53f33b75c3ac3ea8db0d140a68bbf9a5Warning: This coverage report may be inaccurate.This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Details
💛 - Coveralls |
There was a problem hiding this comment.
Pull request overview
This PR restores the inline link feature for synced patterns in WordPress 6.9 by preserving core WordPress link capabilities when registering custom format types.
Changes:
- Modified format registration logic to merge core link settings with custom settings
- Added logic to retrieve and preserve original
core/linksettings before replacing them - Reduced ESLint max-warnings count from 44 to 43
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/js/src/initializers/block-editor-integration.js | Added logic to preserve core link settings when registering custom format types, ensuring WordPress 6.9 compatibility |
| packages/js/package.json | Decreased ESLint max-warnings threshold by 1 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Store the original core/link settings before removing it | ||
| // This ensures we preserve all internal WordPress capabilities and references | ||
| const coreLinkSettings = select( "core/rich-text" ).getFormatType( "core/link" ); | ||
|
|
There was a problem hiding this comment.
The coreLinkSettings retrieval could return undefined if core/link hasn't been registered yet at this point in the execution. This would cause the merge logic on lines 58-60 to silently fail to preserve core capabilities. Consider adding a check to ensure coreLinkSettings exists before proceeding with format registration, or document why this timing is guaranteed to be safe.
| // If the core/link format is not yet registered, do not proceed with | |
| // modifying format types to avoid losing core capabilities. | |
| if ( ! coreLinkSettings ) { | |
| return; | |
| } |
There was a problem hiding this comment.
We are checking if coreLinkSettings on line 58.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
|
LGTM ✅ |
Context
Summary
This PR can be summarized in the following changelog entry:
Relevant technical choices:
Test instructions
Test instructions for the acceptance test before the PR gets merged
This PR can be acceptance tested by following these steps:
Relevant test scenarios
Test instructions for QA when the code is in the RC
QA can test this PR by following these steps:
Impact check
This PR affects the following parts of the plugin, which may require extra testing:
Other environments
[shopify-seo], added test instructions for Shopify and attached theShopifylabel to this PR.[yoast-doc-extension], added test instructions for Yoast SEO for Google Docs and attached theGoogle Docs Add-onlabel to this PR.Documentation
Quality assurance
grunt build:imagesand commited the results, if my PR introduces new images or SVGs.Innovation
innovationlabel.Fixes #22794