New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Site Migration: Show an error message when the site setup fails #90593
Conversation
Jetpack Cloud live (direct link)
Automattic for Agencies live (direct link)
|
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
...ve-flow/internals/steps-repository/site-migration-instructions-i2/pending-actions/style.scss
Outdated
Show resolved
Hide resolved
...ve-flow/internals/steps-repository/site-migration-instructions-i2/pending-actions/style.scss
Outdated
Show resolved
Hide resolved
Aside from comments about color variables, this is working! 👍 |
d866b58
to
a23e87f
Compare
It help us to speed up the test execution, because on error scenarios is required to wait all retry attempts to be completed before see error messages
a23e87f
to
2b81ccb
Compare
This PR modifies the release build for the following Calypso Apps: For info about this notification, see here: PCYsg-OT6-p2
To test WordPress.com changes, run |
efd7d3b
to
8e00f56
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Closes #90228
Proposed Changes
Tech changes
Add configurations to manage the retry attempts and retry delay to optimize the test execution.
Testing Instructions
It is not possible to test using calypso.live because it requires code changes to force errors.
wp-calypso/client/landing/stepper/hooks/use-prepare-site-for-migration.ts
Line 6 in 2a620b6
const PLUGIN = { name: 'migrate-guru-wrong/migrateguru-wrong', slug: 'migrate-guru' };
Pre-merge Checklist