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

ASTField query, not its subqueries #241

Open
keywinf opened this issue Feb 22, 2020 · 1 comment
Open

ASTField query, not its subqueries #241

keywinf opened this issue Feb 22, 2020 · 1 comment

Comments

@keywinf
Copy link

keywinf commented Feb 22, 2020

Hi, thanks for this great repo.

I suggest we get the $ast itself up to the ResolveInfo, not its subfields ($astFields), since we could want to retrieve its location. Example : in a Resolve callback, of a specific field, we could check if user has sufficient rights to read the field, and if not nullify the value and add an error to the ExecutionContext with Location information inside the message (line, column).

I did a fork to get this working, but I'd love to hear about you on this.

Thanks

@Jalle19
Copy link
Collaborator

Jalle19 commented Feb 24, 2020

Feel free to make a PR if you already have it coded up.

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

No branches or pull requests

2 participants