search

graphql / graphql-spec

GraphQL is a query language and execution engine tied to any backend service.
https://spec.graphql.org
14.29k stars 1.12k forks source link

enhance(ResolveFieldValue): note that list items may be async #1066

Open yaacovCR opened 10 months ago

yaacovCR commented 10 months ago

and some baseline collection language for comparison

extracted from #742

netlify[bot] commented 10 months ago

Deploy Preview for graphql-spec-draft ready!

Name Link
Latest commit 32053c0233d872e36d968a38c85bff7b3bab8a0d
Latest deploy log https://app.netlify.com/sites/graphql-spec-draft/deploys/66eb2ba856279f0008d32e04
Deploy Preview https://deploy-preview-1066--graphql-spec-draft.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

yaacovCR commented 10 months ago

@graphql/tsc -- I think this could count as an editorial change? It is currently implemented within graphql-js on main as part of the incremental delivery alpha/beta, has not been released...

yaacovCR commented 10 months ago

Please restore the paragraph breaks which will both minimise the diff and restore the formatting of the note (which has special handling in spec-md).

Done, thanks!

benjie commented 2 weeks ago

@graphql/tsc This PR extends the non-normative note about resolvers being asynchronous to also note that each entry in a list returned from a resolver can itself be asynchronous. Please either approve or express your concerns - thanks!