-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
Mutations with named resolvers #851
Comments
i implemented named resolvers to support batching for optimizing queries. |
A mutation can return any of the same data as a query, so I was surprised to find that they weren't handled the same. (Perhaps some users of GraphQL don't commonly return complex data from mutations? But our application involves a Relay frontend, where a mutation will commonly request all the data necessary to update a page.) Re: side effects, I would think that the only requirement would be to ensure that the mutation itself (the handler for the field of the Happy to hear from you on this! At my work we gave up on using named resolvers after encountering a number of problems with them, and implemented our own somewhat limited batching system instead. If Morpheus is back under active development, maybe we can help resolve some of those issues. |
@sfwc you can open a pr to improve default batching this could help other developers too :) about mutation i think i should rethink and will answer in more details later |
What's currently preventing support for using named resolvers with mutations?
I have a production use case for them and would work on a PR, but want to make sure I'm not barking up the wrong trees.
The text was updated successfully, but these errors were encountered: