You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For defer, it's slightly more complex because we only want to exclude it during the initial execution phase. I am not yet sure how to determine if a resolver is running deferred or not.
@stream may be useful as well for list or connection fields, but would likely be more complicated to implement
The text was updated successfully, but these errors were encountered:
I came across this issue today... I think it would be nice to support at least @skip and @include first, which are the most common used directives. would you mind sharing the place where that early return would be? thanks!
The prisma plugin should support the ~
@skip
! ~@include
! and@defer
directives.For skip and include, we can return early to exclude the fields selection if the field will not be resolved here: https://github.com/graphql/graphql-js/blob/main/src/execution/collectFields.ts#L250
For defer, it's slightly more complex because we only want to exclude it during the initial execution phase. I am not yet sure how to determine if a resolver is running deferred or not.
@stream
may be useful as well for list or connection fields, but would likely be more complicated to implementThe text was updated successfully, but these errors were encountered: