Reapply Site Migration: Default to hiding the migration trial from unverified users #90599
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.
NOTE: We first attempted this in #90589 and ran into weird pre-release test failures. I manually ran the pre-release tests on this PR and they passed so I'll try deploying again on Mon.
Proposed Changes
We've decided to default to hiding the migration free trial from unverified users across all migration flows. This behavior can be overridden by passing the
data
parameter tonavigate
.Testing Instructions
As a logged in a verified user, go to
/start
.Proceed through the migration flow.
On the Upgrade Plan step, verify that you have the option for a 7-day trial.
As a new, unverified user, go to
/start
.Proceed through the migration flow.
On the Upgrade Plan step, verify that you do not have the option for a 7-day trial. You should only see "Upgrade and migrate"/
Pre-merge Checklist