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

SchemaParser#createInputObject issue creating directives #280

Closed
eferreyramontes opened this issue May 28, 2019 · 1 comment
Closed

SchemaParser#createInputObject issue creating directives #280

eferreyramontes opened this issue May 28, 2019 · 1 comment
Milestone

Comments

@eferreyramontes
Copy link

eferreyramontes commented May 28, 2019

There's a small bug regarding inputObject directives in this line guys:

https://github.com/graphql-java-kickstart/graphql-java-tools/blob/master/src/main/kotlin/com/coxautodev/graphql/tools/SchemaParser.kt#L208

Instead of:

.withDirectives(*buildDirectives(definition.directives, setOf(), Introspection.DirectiveLocation.INPUT_FIELD_DEFINITION))

you should have:

.withDirectives(*buildDirectives(inputDefinition.directives, setOf(), Introspection.DirectiveLocation.INPUT_FIELD_DEFINITION))
@oliemansm oliemansm added this to the 5.6.1 milestone Jun 18, 2019
@elielsonms
Copy link
Contributor

elielsonms commented Aug 20, 2019

So, I'm using the 5.5.2 version on my project and I really need this tiny bug resolution on our project. We are avoiding to move to a newer version because that has a graphl-java update and will cause a lot of modifications on our project. Is It possible to generate a 5.5.3 version just including this bug resolution? Well, I just saw that there a 5.5.3 with the Graphl-Java upgraded to 12, so is it possible to have a 5.5.2.1?

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

3 participants