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

Properly handle integer id

Open Sytten opened this issue 6 months ago • 3 comments

Fixes #542

The integer ID parsing IMO should not have been merged at all, but I spend the day fixing it so at least it is fixed now.

For those that dont care about paying the penalty of this feature, I made a feature that you can disable.

Sytten avatar Jun 27 '25 17:06 Sytten

The CI is broken because of unrelated problems

Sytten avatar Jun 27 '25 20:06 Sytten

I think we should consider removing integer ID deserialization support.

tomhoule avatar Jul 07 '25 08:07 tomhoule

Let's do that (https://spec.graphql.org/draft/#sec-ID).

tomhoule avatar Jul 07 '25 08:07 tomhoule