Rob Richard
Rob Richard
@yaacovCR There's a possibility for conflict when you combine defer and stream. I posted an example here: https://gist.github.com/robrichard/0a5e9fb7b0e615545198183c2b9c95a8
@yaacovCR in this proposal, label is an optional argument to both `@defer` and `@stream`, so you will not be required to pass it unless your client library requires it. While...
@IvanGoncharov the RFC changes were already merged here: https://github.com/graphql/graphql-spec/pull/745 I rebased this PR so that change no longer is shown in the diff.
@benjie thanks for taking a look. It is our intention for the spec to support a final payload of `{hasNext: false}` for exactly this type of use case. I will...
@benjie I made some updates to these sections, let me know if there's anything else that's unclear.
@benjie thanks for reviewing this. I think I've either addressed or left notes on all your comments. Please let me know if there's anything I can improve.
@eapache thanks for your review! I've corrected the typos and incorporated your suggestions.
See https://github.com/robrichard/graphql-fields/issues/27
@gienec, I don't plan on adding support for aliases as it cannot be cleanly supported with the current API. I'm working on a replacement library with a new API that...
I think we're mostly on the same page. I think we need a structured class object with methods to access selections as iterables. I don't think we should expose the...