-
Notifications
You must be signed in to change notification settings - Fork 3.3k
internal: (studio) quality of life improvements for the studio lifecycle #31663
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
internal: (studio) quality of life improvements for the studio lifecycle #31663
Conversation
cypress
|
Project |
cypress
|
Branch Review |
ryanm/internal/provide-more-info-on-studio-session-failures
|
Run status |
|
Run duration | 19m 25s |
Commit |
|
Committer | Ryan Manuel |
View all properties for this run ↗︎ |
Test results | |
---|---|
|
0
|
|
8
|
|
1232
|
|
0
|
|
32157
|
View all changes introduced in this branch ↗︎ |
UI Coverage
46.24%
|
|
---|---|
|
188
|
|
166
|
Accessibility
92.74%
|
|
---|---|
|
3 critical
9 serious
2 moderate
2 minor
|
|
889
|
@@ -47,6 +46,10 @@ const downloadStudioBundleToTempDirectory = async ({ studioUrl, projectId }: Opt | |||
encrypt: 'signed', | |||
}) | |||
|
|||
if (!response.ok) { | |||
throw new Error(`Failed to download studio bundle: ${response.statusText}`) |
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.
What's the user experience with throwing this error?
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.
We catch it up stream and report it. I'll take a look at @astone123's PR to make sure it plays nicely there, but prior to this change it would actually have failed in a different (more confusing) way.
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
…ents for the studio lifecycle (#31663)
…ts for the studio lifecycle (#31663)
* chore: updating v8 snapshot cache * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) --------- Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
* chore: updating v8 snapshot cache * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) --------- Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
* chore: updating v8 snapshot cache * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) * index on develop: e3a7994 internal: (studio) quality of life improvements for the studio lifecycle (#31663) --------- Co-authored-by: cypress-bot[bot] <+cypress-bot[bot]@users.noreply.github.com>
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
Additional details
This PR adds a few quality of life improvements for the Studio initialization workflow. It provides additional debugging context when things fail.
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation
?type definitions
?