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

docs(readme): add missing required email in insert examples #448

Merged
merged 1 commit into from Jul 27, 2022

Conversation

nesb1
Copy link
Contributor

@nesb1 nesb1 commented Jul 25, 2022

Change Summary

Hello.
In example schema.prisma file User has email required field. But in example with python client specified only name. If run it example than an error will occur:
prisma.errors.MissingRequiredValueError: Failed to validate the query: `Unable to match input value to any allowed input type for the field. Parse errors: [Query parsing/validation error at `Mutation.createOneUser.data.UserCreateInput.email`: A value is required but not set., Query parsing/validation error at `Mutation.createOneUser.data.UserUncheckedCreateInput.email`: A value is required but not set.]` at `Mutation.createOneUser.data

Checklist

  • Unit tests for the changes exist
  • Tests pass without significant drop in coverage
  • Documentation reflects changes where applicable
  • Test snapshots have been updated if applicable

Agreement

By submitting this pull request, I confirm that you can use, modify, copy and redistribute this contribution, under the terms of your choice.

@nesb1 nesb1 changed the title Added missing required email in insert examples (fix) Added missing required email in insert examples Jul 25, 2022
@RobertCraigie RobertCraigie changed the title (fix) Added missing required email in insert examples docs(readme): add missing required email in insert examples Jul 27, 2022
@RobertCraigie
Copy link
Owner

Thank you for spotting this!

@RobertCraigie RobertCraigie merged commit e0c51cd into RobertCraigie:main Jul 27, 2022
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.

None yet

2 participants