graphql / graphql/graphql.github.io
learn/execution resolver function - should mention signature is specific to js server
Personne n'a encore pris cette issue.
- Langage dominant
- TypeScript
- Étoiles
- 889
- Forks
- 1.5k
- Merge moyen
- 4 j 12 h
- PR mergées (30 j)
- 21
Description
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.
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Ouvrez open source/src/content/learn/Learn-Execution.md et trouvez l’affirmation selon laquelle une fonction resolver reçoit quatre arguments. Examinez le contexte lié de la spécification GraphQL, puis mettez à jour la formulation afin d’indiquer clairement que cette signature à quatre arguments est spécifique au serveur JavaScript. Le travail est terminé lorsque la documentation distingue correctement ce comportement d’implémentation de la spécification.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- graphql
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 55/100