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

Ignore items in lists where the type is unknown

Open A360JMaxxgamer opened this issue 2 years ago • 3 comments

Ignores all items when a list is deserialized where the GraphQL type is not part of the schema.graphql file.

  • Add TryParse method to IEntityIdSerliazer
  • Add check whether entity id can be parsed and whether the item shall be added to the list

Closes #6415

A360JMaxxgamer avatar Aug 22 '23 16:08 A360JMaxxgamer

@PascalSenn is it worth fixing the unit tests or is that as designed?

A360JMaxxgamer avatar Aug 22 '23 16:08 A360JMaxxgamer

what is the status on this one?

michaelstaib avatar Aug 24 '23 09:08 michaelstaib

I changed the code to use try parse mechanism. If that solution is generally ok with you, I will fix the rest of the unittests. Basically update the snapshots.

A360JMaxxgamer avatar Aug 24 '23 11:08 A360JMaxxgamer