-
Notifications
You must be signed in to change notification settings - Fork 2
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
feat(errors): show error messaging as banner on failed submissions #474
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.
Test drove it, looks great. It caught a (deliberately imposed) throw buried in the submit function. Nice red banner saying something went wrong. Only comment was scroll-to-top would be good, and I think you've already added or will add that.
Excellent stuff
🎉 This PR is included in version 1.5.0-val.24 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Purpose
To introduce a banner that displays errors provided by the API when submitting a form.
Linked Issues to Close
https://qmacbis.atlassian.net/browse/OY2-27443
Approach
Use our already established alert context to display relevant info from within our catch block on submission.
Assorted Notes/Considerations/Learning
Brian's work setting up the error banner display tooling in the new actions pages made it the quickest change despite the learning curve from the old verbose pattern. Worth it in my opinion!