Fix "configure an AI connector" link on Settings page to point to the Connectors screen#1358
Merged
ernilambar merged 1 commit intoJun 13, 2026
Conversation
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
davidperezgar
approved these changes
Jun 13, 2026
ernilambar
approved these changes
Jun 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What?
Closes: 1357
Fixes the "configure an AI connector" link on the Plugin Check Settings page so it points to the WordPress core Connectors screen (
options-connectors.php) instead of General Settings (options-general.php).Why?
On Settings → Plugin Check, when no AI connector is configured, the AI Code Review section shows this notice:
The link currently points to
admin_url( 'options-general.php' )(Settings → General), which contains no AI/connector settings — users following it land on an unrelated screen with no way forward, making the AI Code Review feature hard to enable.The intended target is the core Connectors screen, which this plugin already links to correctly in
includes/Admin/Namer_Page.php(the "Configure AI Connectors" button usesadmin_url( 'options-connectors.php' )). The translator comment on the string ("URL to WordPress AI settings") also confirms the AI settings screen was the intended destination.How?
One-line change in
includes/Admin/Settings_Page.php(the notice rendered inthe AI Code Review section):
This matches the existing, correct usage in
Namer_Page.php. No stringchanges, so there is no translation impact. The other two
options-general.phpreferences in the codebase are intentionally untouched:the
add_submenu_page()parent slug inSettings_Page.phpand thesettingsPageUrl(with?page=plugin-check-settings) inAdmin_Page.phpboth correctly refer to Plugin Check's own settings page.
Testing Instructions
link inside the warning notice.
(
options-general.php), which has no connector settings.After this PR: the link opens the Connectors settings screen
(
options-connectors.php), consistent with the "Configure AI Connectors"button on the Plugin Namer page.
AI Usage Disclosure
If AI tools were used, please describe how they were used:
Screenshots or screencast