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

add viewer to mutationadd #56

Merged
merged 1 commit into from
Feb 6, 2017
Merged

Conversation

sibelius
Copy link
Collaborator

@sibelius sibelius commented Feb 5, 2017

fix #37

@codecov-io
Copy link

codecov-io commented Feb 5, 2017

Codecov Report

Merging #56 into master will not impact coverage.

@@           Coverage Diff           @@
##           master      #56   +/-   ##
=======================================
  Coverage   82.67%   82.67%           
=======================================
  Files          13       13           
  Lines         202      202           
=======================================
  Hits          167      167           
  Misses         35       35

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c055be4...7212a0b. Read the comment docs.

@rturk
Copy link

rturk commented Feb 5, 2017

This will always include a viewer in the mutation?

@sibelius
Copy link
Collaborator Author

sibelius commented Feb 5, 2017

only on add mutation, we can move this as a parameter or as a config later

@lucasbento
Copy link
Collaborator

@sibelius: The boilerplate has a ViewerLoader, should we use it or not?

@sibelius
Copy link
Collaborator Author

sibelius commented Feb 6, 2017

neh, just return the logged user

@lucasbento lucasbento merged commit c8afb13 into master Feb 6, 2017
@lucasbento lucasbento deleted the feature/viewerMutationAdd branch February 6, 2017 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Return viewer on MutationAdd template
4 participants