-
Notifications
You must be signed in to change notification settings - Fork 887
fix: handle workspace errors #3341
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
Conversation
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.
The code looks great! Do you know why the Chromatic check is being skipped?
@presleyp I published the stories manually here: https://624de63c6aacee003aa84340-uboeickwbe.chromatic.com/?path=/story/components-workspace--error |
Storybook makes it look like we're displaying an error message when a build is successfully canceled - is it supposed to be an error saying it failed to cancel? Or a success message about cancellation? |
This PR handles the errors shown on the Workspace (singular) page. We handle the following types of errors:
Subtasks
ErrorSummary
component for the listed errorsFixes #3094
Screenshots
get resources
get builds
build cancellation
build error
get workspace