aPureBase / KGraphQL

Pure Kotlin GraphQL implementation
https://kgraphql.io
MIT License
301 stars 58 forks source link

InputValue KType #157

Closed ESchouten closed 3 years ago

ESchouten commented 3 years ago

In order to support generic input types discussed in #143, and overcome the jvm erasure issue discussed in #144, kType is added to inputValue.

The code does compile, tests do run, however, whenever I build and use it in my application, it does compile but I get java.lang.NoSuchMethodErrors when trying to invoke these altered methods.

I can't seem to figure out whether it's a caching issue on my side or something else. Could you take a look?

jeggy commented 3 years ago

This looks okay. I will publish this in version 0.17.12 and test it out in a big production app I have, if it runs smoothly there, I think everything should be fine :+1:

Otherwise, I would love a failing unit test if something is wrong.