[graphql-js] separate execute/graphql functions that support AsyncGenerator responses #14
robrichard
announced in
Architectural Decision Records
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Context
The current api changes to graphql-js to support defer/stream introduce a breaking change in the return type of the
execute
andgraphql
functions. The return type changes from returning a Promise or value, to returning either an AsyncGenerator, Promise, or value.We previously implemented separate new functions that support the new return type, preserving the return type in the existing functions. However, this led to a lot of code duplication so this was abandoned.
Additional discussion: graphql/graphql-js#2848 (comment)
Beta Was this translation helpful? Give feedback.
All reactions