Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

learn/execution resolver function - should mention signature is specific to js server #1141

Open
tonyhallett opened this issue Oct 19, 2021 · 0 comments

Comments

@tonyhallett
Copy link

From https://github.com/graphql/graphql.github.io/blob/source/src/content/learn/Learn-Execution.md

A resolver function receives four arguments

I suggest that this is changed to "This resolver function receives four arguments"

Given that the specifications specifies

ResolveFieldValue(objectType, objectValue, fieldName, argumentValues)
Let resolver be the internal function provided by objectType for determining the resolved value of a field named fieldName.
Return the result of calling resolver, providing objectValue and argumentValues.

So although the resolver function with 4 arguments is specification-compliant a resolver function does not receive four arguments. Only the first two.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants