Plans Grid: fix simplified display behaviour #97490
Merged
+8
−3
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.
Fixes #97489
Proposed Changes
This is a follow-up to #95977.
The simplified plans grid is now the default and should be shown in all plans grids. Some variations of the plans grid, specifically with the
'plans-newsletter', 'plans-link-in-bio', 'plans-blog-onboarding'
intents, display a different set of features. These intents and flows will soon be removed. (p5uIfZ-ggB-p2#comment-24253).The previous check implemented in #95977 was too restrictive and resulted in the "Everything in {previous_plan}, plus:" text being shown for plans grids with intents other than the default (
plans-default-wpcom
) and the ones listed above.This PR fixes it by changing the check to ensure the simplified grid is shown in all cases except when the intents are
'plans-newsletter', 'plans-link-in-bio', 'plans-blog-onboarding'
.Why are these changes being made?
Testing Instructions
/domains/manage/all/<domain>/edit/<domain>
for a domain-only purchase./start
,/setup/newsletter/plans
and/plans/<site slug>
to ensure that they are unchanged from production.Pre-merge Checklist