search

graphql-go / relay

A Go/Golang library to help construct a graphql-go server supporting react-relay.
MIT License
424 stars 58 forks source link

Include `error` return value to satisfy updated FieldResolveFn type #11

Closed piinecone closed 8 years ago

piinecone commented 8 years ago

This is basically identical to #9 except it includes updates to the connection and plural test cases and the starwars example's schema. Thanks to @bsr203 from whom I lifted the config.Name addition in mutation.go.

As @bsr203 mentions, this introduces compatibility with https://github.com/graphql-go/graphql/commit/344abcc936d3a1f4b288c5c4609377861d1649c5.

sogko commented 8 years ago

Hi @piinecone

Thanks for this PR! I'll merge this over #9 and #10

Regarding returning the appropriate error messages, we'll need to go through it again in another commit /PR

Thanks again! Cheers!