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

chore: updates user query & createNote mutation #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mrpineapples
Copy link

@mrpineapples mrpineapples commented Feb 22, 2020

  • While both queries work I think changing OpName to User is clearer since it's finding a single user.
  • In CreateNote the note is currently being appended to every user, this is fixed by checking that the passed in ID matches the user's ID before appending.

@mrpineapples mrpineapples changed the title chore: updates user query chore: updates user query & createNote mutation Feb 22, 2020
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.

1 participant