Support latest graphql-js about graphql-sequelize HOT 5 CLOSED

Thanks, i will look into it :)

from graphql-sequelize.

I've looked into this a bit more. The resolve function signature changed in 0.3.0.

Field resolve functions now have a new API: resolve(value, args, info), where info is an object which contains the same information which used to be separate arguments, and includes more useful stuff! https://github.com/graphql/graphql-js/releases/tag/v0.3.0

I've done this in the meantime:

let photoResolver = resolver(Photo);
//...
resolve: (value, args, {rootValue, fieldASTs, returnType}) => {
  return photoResolver(value, args, rootValue, fieldASTs, returnType);
}
//...

from graphql-sequelize.

Ah okay so they mixed root, field and type into one field, should be a pretty trivial fix. Changing https://github.com/mickhansen/graphql-sequelize/blob/master/src/resolver.js#L31 to source, args, {root, ast, type} should work i think.

I'll see if i can't get a fix in today.

Thanks for using this lib @amccloud :)

from graphql-sequelize.

Hmm, quite a few changes in how the AST is passed around now, working on a fix.

from graphql-sequelize.

Should now work in [email protected] do make sure to thoroughly test it though.
fieldASTs is now an array with a single element instead of the single element so i'm doing a naive fix with fieldASTs[0] but all my tests pass (but it's not the most comprehensive test suite in the world).

from graphql-sequelize.