smallrye-graphql
smallrye-graphql copied to clipboard
#1423: add GraphQLResult and wrap successful result
Also: rename Wrapper#isNotEmpty to Wrapper#isNonNull and fix some warnings
TODOs:
- [ ] Map all declared exceptions to fields
- [ ] In the schema
- [ ] When executing
- [ ] Consider
@GraphQLResultannotations on an API class - [ ] Consider
@GraphQLResultannotations on a package - [ ] Provide a config option to globally enable result-wrapping (and add to
GraphQLResultjavadoc) - [ ] Maybe consider ErrorCode annotation (but convert kebap to camel?)
- [ ] Documentation
@phillip-kruger: I have several smaller problems that you might be able to help with.
- The
mainbranch doesn't compile (GraphQLWSSubprotocolHandlerandGraphQLTransportWSSubprotocolHandler) - In the
mainbranch, theReferenceCreatorcauses a StackOverflow - I've added a TCK test for
result, but it fails because it doesn't find theordermethod, but the same request runs in the runner!
I could fix 1. and 2., but I don't understand how they got in there.