graphqlError: missing fields

[bsr203]

Hi,

I was trying to extend the graphql error to have custom error codes. In this blog post, Apollo engineer explains how they use extensions field for that. I don't use Apollo Server, so don't know currently how they do it. Still, the error type in reason-apollo doesn't define all fields.

currently it is defined as

type graphqlError = {
  .
  "message": string,
  "locations": Js.Nullable.t(array(string)),
  "path": Js.Nullable.t(array(string)),
  "nodes": Js.Nullable.t(array(string)),
};

Apollo client defines it here as

graphQLErrors?: GraphQLError[];

GraphQLError is defined at graphql-js as

  message: string,
    nodes?: $ReadOnlyArray<ASTNode> | ASTNode | void,
    source?: ?Source,
    positions?: ?$ReadOnlyArray<number>,
    path?: ?$ReadOnlyArray<string | number>,
    originalError?: ?Error,
    extensions?: ?{ [key: string]: mixed },

It will be great to have extensions and the other missing fields.

thanks.

[fakenickels]

would you mind sending us a PR including those?

[bsr203]

sorry. I am not using this library anymore as decided to stick with typescript. Please close this if it is not affected by other users.