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

Support custom Ids with Global Object Identification

Open tobias-tengler opened this issue 2 years ago • 3 comments

Todos

  • Correctly use type converter on output fields and fix failing test
  • Correctly handle filtering

Closes #5114 #4522

tobias-tengler avatar Jun 15 '22 11:06 tobias-tengler

@PascalSenn would be great if you could give this a look. :) I'm not sure how to properly use the Type converter in combination with the output field ResultConverters to get the desired effect and I'm also not sure if I'm using the Type converter in the correct place for the argument list of ids.

tobias-tengler avatar Jun 15 '22 11:06 tobias-tengler

The whole formatter class is so complicated :) I will have another look tomorrow ... I think we could reduce this to just a couple of lines.

michaelstaib avatar Jun 15 '22 22:06 michaelstaib

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Aug 17 '22 14:08 CLAassistant