Fix Blueprint Version Warning Comparison - #2494
Merged
sejas merged 3 commits intoJan 28, 2026
Merged
Conversation
Simplify the blueprint preferred versions warning logic: - Validate WP versions against the API-fetched supported versions list - Compare blueprint versions to current user selection instead of original defaults - Update warning text from "default is" to "selected is" for clarity - Remove unnecessary originalDefaultVersions prop
Collaborator
📊 Performance Test ResultsComparing f147d0c vs trunk site-editor
site-startup
Results are median values from multiple test runs. Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change |
epeicher
approved these changes
Jan 28, 2026
epeicher
left a comment
Contributor
There was a problem hiding this comment.
Thanks @sejas! This works as expected. I can see validation warnings when using versions different from those specified in the Blueprint. The WordPress version is automatically updated to latest when it is unsupported, while the unsupported php version is just discarded.
Also the warnings are displayed as soon as the versions of PHP or WordPress is changed to a different one than selected in the Blueprint. LGTM!
| Incorrect WP version collapsed | Incorrect WP version expanded | Selecting different version than the Blueprint | Non-supported PHP version |
|---|---|---|---|
![]() |
![]() |
![]() |
![]() |
sejas
deleted the
stu-1265-ensure-the-warning-in-the-blueprint-advanced-settings
branch
January 28, 2026 16:11
ivan-ottinger
left a comment
Contributor
There was a problem hiding this comment.
Thank you for the fix, Antonio! The changes work nicely!
CleanShot.2026-01-29.at.09.58.22.mp4
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
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.




Related issues
Proposed Changes
Testing Instructions
npm startTest 1: Warning appears when blueprint version uses unsupported WP version
blueprint-unsupported-wp.json
preferredVersions(e.g., WP 5.8.1)Test 2: Warning updates dynamically when user changes dropdown
test-multiple-warnings-blueprint.json
Test 3: No supported PHP version
blueprint-unsupported-php.json
We didn't modify the logic, but I observed that if the PHP version is not supported, the flow displays an error validating the Blueprint in the previous step.
Pre-merge Checklist