add support for GraphQL Docs Request for client request, add fix can'…#18
add support for GraphQL Docs Request for client request, add fix can'…#18yongboy wants to merge 1 commit intobjornbytes:masterbjornbytes/graphql-lua:masterfrom
Conversation
…t get the InputObject's Parameter bug, and fix the directive does not work well bug.
| resolve = function(kind) | ||
| if kind.__type == 'Object' then | ||
| return kind.interfaces | ||
| return kind.interface or {} |
There was a problem hiding this comment.
Shouldn't this still be .interfaces?
| variableMap[_vn] = true | ||
| -- TODO Maybe danger | ||
| if not _cv[_vn] then | ||
| _cv[_vn] = true |
There was a problem hiding this comment.
This leads to a regression where unused variables don't fail validation anymore, try validating query test($arg: String) {}.
|
Any updates on this @yongboy? |
|
some of the changes are integrated here (PR will come later). https://github.com/ruslantalpa/graphql-lua/tree/deploy the docs work for me |
|
I'm not sure if this thread is dead, but I've got this working with GraphiQL with the following minimal patch: andyvanee@4785941 Also, I was a bit confused since the only available version on luarocks is the 0.0.2 tag, which is quite old and does not include many of the changes necessary to work with GraphiQL - particularly 119aafe . Perhaps it's time for an updated tag? Thanks! |
Fix bugs:
Docsfailure:))