You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The graphql documentation https://graphql.org/learn/schema/ defines Int as A signed 32‐bit integer. . We recently ran into a problem where we sent a value that exceed the 32 bit maximum on our server which caused an error. Our server expects 32bit integers.
Describe the solution you'd like
An option added to ApolloCodegenConfiguration.OutputOptions which specifies whether Int should be explicitly Int32, Int64, or Int (variable based on platform).
The text was updated successfully, but these errors were encountered:
Hi @scottasoutherland - thanks for creating the issue, I can sense the confusion this must have caused.
I've spoken with the rest of the team and we agree this should be a configuration option for the Swift generated code. The codegen configuration default for 1.0 should remain as it is now though (Int) to avoid a breaking change for everyone. Though in the upcoming 2.0 release the codegen configuration default should change to be Int32 to better match the GraphQL spec as default.
Our next planning meeting isn't for a while so I haven't prioritized it yet but if you are interested in contributing we can certainly work together on the configuration option and implementation.
Use case
The graphql documentation https://graphql.org/learn/schema/ defines Int as
A signed 32‐bit integer.
. We recently ran into a problem where we sent a value that exceed the 32 bit maximum on our server which caused an error. Our server expects 32bit integers.Describe the solution you'd like
An option added to
ApolloCodegenConfiguration.OutputOptions
which specifies whetherInt
should be explicitly Int32, Int64, or Int (variable based on platform).The text was updated successfully, but these errors were encountered: