@GraphQLID support for List<ID>

I might be missing something—is it possible to use the @GraphQLID annotation on a List<String>?

class Query {
  fun thingsByID(@GraphQLID ids: List<String>): List<Thing> = ...

When version 2.1.1, I get:

com.expediagroup.graphql.exceptions.InvalidIdTypeException: List is not a valid ID type, only Strings are accepted

I have a workaround, but I wanted to check to see if I was just doing something wrong:

typealias ID = String

class Query {
  fun thingsByID(ids: List<ID>): List<Thing> = ...
}

class CustomSchemaGeneratorHooks(
  federatedTypeRegistry: FederatedTypeRegistry
) : FederatedSchemaGeneratorHooks(federatedTypeRegistry) {
  override fun willGenerateGraphQLType(type: KType): GraphQLType? = when (type.classifier as? KClass<*>) {
    ID::class -> Scalars.GraphQLID
    else -> super.willGenerateGraphQLType(type)
  }
}

Thanks!

[christianrybe]

wow, fantastic guys, thanks a ton for making this happen so quickly, I’ve just hit this issue with the previous version and I’m now happily back to my work using the 3.0 RC 😃

[lennyburdette]

Awesome! Not expecting a blazing fast response!

I’m all for a wrapper type if that’s the direction you decide to go. I had a few moments to try to support the annotation and I think I have it working: https://github.com/ExpediaGroup/graphql-kotlin/compare/master...lennyburdette:lenny/support-lists-of-ids?expand=1

But I agree that passing the annotatedAsId around everywhere isn’t fun.