Skip to content

graphql: default variables are not working #866

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.

Sign up for GitHub

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

Jump to bottom
Closed
RunsFor opened this issue Jun 2, 2020 · 0 comments · Fixed by #920
Closed

graphql: default variables are not working #866

RunsFor opened this issue Jun 2, 2020 · 0 comments · Fixed by #920
Labels
bug Something isn't working graphql

Comments

@RunsFor
Copy link
Contributor

RunsFor commented Jun 2, 2020

Two screenshot below should be self-explanatory. First query without default variable:

Screenshot 2020-06-02 at 11 49 31

Second with uuid default variable:

Screenshot 2020-06-02 at 11 50 17

One server record were expected to be in response
@olegrok olegrok added bug Something isn't working graphql labels Jun 2, 2020
olegrok added a commit that referenced this issue Jun 29, 2020
@olegrok
graphql: consider default values for graphql execution
526983c 
Closes #866
@olegrok olegrok mentioned this issue Jun 29, 2020
Merged
olegrok added a commit that referenced this issue Jun 30, 2020
@olegrok
graphql: consider default values for graphql execution
2d4e8de 
Note: there is a difference between explicitly passed "null" value
and just omitted variable. See graphql/graphql-spec#418

Closes #866
@rosik rosik closed this as completed in #920 Jul 2, 2020
rosik pushed a commit that referenced this issue Jul 2, 2020
@olegrok
graphql: support default variables (#920)
6eb4a43 
See: https://graphql.org/learn/queries/#default-variables
Note: there is a difference between explicitly passed "null" value
and just omitted variable. See graphql/graphql-spec#418

Closes #866
olegrok added a commit to tarantool/graphql that referenced this issue Feb 11, 2021
@olegrok
support default variables
470d4c3 
Imported from tarantool/cartridge@6eb4a43

```
See: https://graphql.org/learn/queries/#default-variables
Note: there is a difference between explicitly passed "null" value
and just omitted variable. See graphql/graphql-spec#418

Closes tarantool/cartridge#866
```
olegrok added a commit to tarantool/graphql that referenced this issue Feb 11, 2021
@olegrok
support default variables
5963111 
Imported from tarantool/cartridge@6eb4a43

```
See: https://graphql.org/learn/queries/#default-variables
Note: there is a difference between explicitly passed "null" value
and just omitted variable. See graphql/graphql-spec#418

Closes tarantool/cartridge#866
```
olegrok added a commit to tarantool/graphql that referenced this issue Feb 25, 2021
@olegrok
support default variables
93ba254 
Imported from tarantool/cartridge@6eb4a43

```
See: https://graphql.org/learn/queries/#default-variables
Note: there is a difference between explicitly passed "null" value
and just omitted variable. See graphql/graphql-spec#418

Closes tarantool/cartridge#866
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working graphql
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

graphql: consider default values for graphql execution tarantool/cartridge
2 participants
@RunsFor @olegrok