Added operationName
#58
Conversation
ghost
added
the
| { | ||
| "query": "...", | ||
| "variables": { variable1: value, ... } | ||
| "oprationName": "...", |
There was a problem hiding this comment.
I think you're missing the e in operationName here and also above.
Also, I think it's worth mentioning that the operationName in the POST body is optional, and only required if you have multiple operations?
As an additional wrinkle, express-graphql doesn't support multiple operations in one document as far as I know.
There was a problem hiding this comment.
OMG, sorry for the typo 😳
I was also thinking about adding a note about optional fields. In this case we need to mention that variables is optional as well. WDYT?
There was a problem hiding this comment.
Yeah, maybe just add a sentence underneath the code snippet saying operationName and variables are optional.
There was a problem hiding this comment.
Perfect. I already added it and rebased the PR
|
Just one last question - should we be worried that |
|
Are you sure about this limitation? I just double checked the docs and looks like |
|
Oh, OK I'm totally wrong :] |
2 participants
No description provided.