Skip to content

Conversation

@ivan-ottinger
Copy link
Contributor

@ivan-ottinger ivan-ottinger commented Jan 22, 2026

Related issues

Proposed Changes

  • add user-friendly error messages when Blueprint deeplink open fails
  • add related tests
Before After
CleanShot 2026-01-23 at 10 53 59@2x CleanShot 2026-01-23 at 11 02 05@2x
CleanShot 2026-01-23 at 10 49 15@2x CleanShot 2026-01-23 at 11 01 46@2x

Testing Instructions

  1. Check out the PR branch and build the app with npm install && npm start.
  2. Try opening first two broken Blueprint deeplinks at https://jsbin.com/kulaqijega/1/edit?html,css,output.
  3. Both should open an user-friendly error message with option to see the exact error message in the Studio app logs.

Pre-merge Checklist

  • Have you checked for TypeScript, React or other console errors?

@ivan-ottinger ivan-ottinger self-assigned this Jan 22, 2026
@ivan-ottinger ivan-ottinger force-pushed the update/blueprint-deeplink-error-messages branch from 0a1bc13 to 9c76a87 Compare January 23, 2026 10:04
@ivan-ottinger ivan-ottinger marked this pull request as ready for review January 23, 2026 10:05
@ivan-ottinger ivan-ottinger requested a review from a team January 23, 2026 10:05
Copy link
Contributor

@epeicher epeicher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ivan-ottinger for improving this! I have tested it, and I can see the new error messages.

Although I'm not sure I like the double Failed to load Blueprint and then The Blueprint could not be loaded. and not displaying the returned error message. I would be interested in others' views on those

Network Error Invalid Blueprint error
Image Image

detail: expect.any( String ),
buttons: expect.any( Array ),
message: 'Failed to load Blueprint',
detail: 'The Blueprint could not be loaded. Please check the link and try again.',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree with Roberto's assessment regarding the wording being repetitive. Maybe The Blueprint could not be loaded could be skipped altogether.

@katinthehatsite
Copy link
Contributor

Although I'm not sure I like the double Failed to load Blueprint and then The Blueprint could not be loaded. and not displaying the returned error message. I would be interested in others' views on those

I agree with this assessment, although I think we don't necessarily need the exact error message in this window as as it will be in the logs as well. We should definitely adjust the repetitive wording though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants