graphql-platform icon indicating copy to clipboard operation
graphql-platform copied to clipboard

Support lists in external reference resolvers

Open luke-pp opened this issue 11 months ago • 1 comments

Add support for the Apollo [External] attribute/directive on list types.

  • Currently the ArgumentParser class does not support list types, meaning the reference resolver always receives null for the external list type even when a list is provided to it (see #7991 for details)
  • This PR adds a test to the existing suite for the @external support, and an implementation for the ArgumentParser that lets it parse lists.
  • The implementation is based on the existing implementation for the InputParser, although the function signature is a bit different
  • There are a couple of edge cases that still need implementation, I'll need a bit of guidance to make sure they work as intended

Closes #7991

luke-pp avatar Feb 06 '25 18:02 luke-pp

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Feb 06 '25 18:02 CLAassistant