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.

Sign up for GitHub

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

Jump to bottom

Fix createMany Input Type requiring an id #141

Merged
merged 1 commit into from
Jun 15, 2022
Merged

Fix createMany Input Type requiring an id #141

merged 1 commit into from
Jun 15, 2022

Conversation

slax57
Copy link
Collaborator

@slax57 slax57 commented Jun 15, 2022 •
edited

Description

Removes the id field from the *Input type (used only for the createMany mutation).

Related Issue

Fixes #132

ToDo

  • Fix createMany Input Type requiring an id although it is generated

@slax57 slax57 added the WIP label Jun 15, 2022
@slax57 slax57 force-pushed the 132-create-mutation-does-not-require-id-but-createmany-mutation-does branch from a1491d5 to 625050b Compare June 15, 2022 16:31
@slax57 slax57 added RFR Ready For Review and removed WIP labels Jun 15, 2022
@slax57 slax57 added this to the 2.3.2 milestone Jun 15, 2022
@fzaninotto fzaninotto merged commit 1c5690e into master Jun 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fzaninotto fzaninotto fzaninotto approved these changes

@antoinefricker antoinefricker antoinefricker approved these changes

Assignees
No one assigned
Labels
RFR Ready For Review
Projects
None yet
Milestone
2.3.2
Development

Successfully merging this pull request may close these issues.

Create mutation does not require ID, but CreateMany mutation does
3 participants
@slax57 @fzaninotto @antoinefricker