Skip to content
This repository was archived by the owner on Apr 14, 2022. It is now read-only.

Support directives: @skip, @include #45

Closed
Totktonada opened this issue Feb 9, 2018 · 0 comments
Closed

Support directives: @skip, @include #45

Totktonada opened this issue Feb 9, 2018 · 0 comments
Assignees
Labels
enhancement New feature or request prio1

Comments

@Totktonada
Copy link
Member

Totktonada commented Feb 9, 2018

It seems that grapql-lua supports directives already, so we need to write a test to check all works as expected. The example can be found here: http://graphql.org/learn/queries/#directives . I think the test can reuse the common_testdata dataset and conditionally include/skip a connection field. I think we need to check only a boolean variable as the expression of a directive and postpone more complex cases when it will be requested explicitly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request prio1
Projects
None yet
Development

No branches or pull requests

2 participants