-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Improve error messages for validation failures #6218
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/keystonejs/keystone-next-docs/Fh8CAkfimkvhEjDCQjkeeJwKkLU3 |
🦋 Changeset detectedLatest commit: cc90f8a The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. |
a89ea23
to
02900c3
Compare
29c3d3c
to
0ae205c
Compare
0ae205c
to
b5d2702
Compare
b5d2702
to
3e7b008
Compare
3e7b008
to
4bf7a5b
Compare
7735ad2
to
0398125
Compare
0398125
to
4ba0286
Compare
4ba0286
to
ca0c68e
Compare
ca0c68e
to
c979cd8
Compare
This PR updates the error message provided during the
validation
phase of our CRUD mutations. The changes are:isRequired
, field hooks, list hooks) are now run to completion before returning the errors, so as to provide as much feedback as possible to the user.Config
Before
After