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

Add support of fragments' arguments #89

Closed
SudoBobo opened this issue Mar 16, 2018 · 2 comments
Closed

Add support of fragments' arguments #89

SudoBobo opened this issue Mar 16, 2018 · 2 comments
Labels
enhancement New feature or request prio2

Comments

@SudoBobo
Copy link
Contributor

Currently they are not supported by graphql-lua. This will NOT work:

... on Image (size: small) {
    resolution
    uploadedAt
}

It reduces feature capacity of union connections in compare with 'single' ones, as fragments are the only way to access union fields. See graphql/graphql-spec#204

@SudoBobo SudoBobo added enhancement New feature or request prio2 labels Mar 16, 2018
Totktonada pushed a commit that referenced this issue Mar 16, 2018
Fixes #8.

Backlog / related: #84, #85, #86, #88, #89 + some refactoring / code
deduplication.
@SudoBobo SudoBobo self-assigned this Apr 2, 2018
@Totktonada
Copy link
Member

I think we can add arguments to 'box field' in order to support arguments for multi-head connections.

@Totktonada
Copy link
Member

I'll close it as dupe of more general #185.

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

No branches or pull requests

2 participants