Skip to content
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

fix(errors) : handle missing types etc #352

Merged
merged 1 commit into from
Jul 14, 2021
Merged

Conversation

dselman
Copy link
Contributor

@dselman dselman commented Jul 9, 2021

Signed-off-by: Dan Selman [email protected]

Improves error handling for form generation so that the storybook doesn't crash when we make changes to the model, the name of the type to generate etc.

To test, try adding a new model file to the Knobs and recreating the form.

@dselman dselman requested a review from mttrbrts July 9, 2021 12:15
@cypress
Copy link

cypress bot commented Jul 9, 2021



Test summary

1 0 0 0


Run details

Project web-components
Status Passed
Commit 0d07ed9
Started Jul 9, 2021 12:18 PM
Ended Jul 9, 2021 12:19 PM
Duration 01:21 💡
OS Linux Ubuntu - 20.04
Browser Chrome 91

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@cypress
Copy link

cypress bot commented Jul 9, 2021



Test summary

1 0 0 0


Run details

Project web-components
Status Passed
Commit df4da9b ℹ️
Started Jul 9, 2021 12:19 PM
Ended Jul 9, 2021 12:20 PM
Duration 01:20 💡
OS Linux Ubuntu - 20.04
Browser Chrome 91

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@mttrbrts mttrbrts merged commit f2450e6 into master Jul 14, 2021
@jolanglinais jolanglinais deleted the ds-error-handling branch November 30, 2021 21:13
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.

2 participants