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
Looks like there is still some old code in the "social" example. If that is the case, is is possible to clean that? It might be confusing for people starting with graphql-go and trying to educate by example (like me) ;-)
@marcelvanwaaijen I haven't looked into it in details but my wild guess is that it was an existing example and when this feature was implemented we ended up adding graphql.UseFieldResolvers() which uses fields instead of methods. Then we started using the fields and stopped using the method. I would suggest opening a separate issue if this is the case as this one was related to adding the field resolvers feature, which is already done.
Looks like there is still some old code in the "social" example. If that is the case, is is possible to clean that? It might be confusing for people starting with graphql-go and trying to educate by example (like me) ;-)
@marcelvanwaaijen I haven't looked into it in details but my wild guess is that it was an existing example and when this feature was implemented we ended up adding
graphql.UseFieldResolvers()
which uses fields instead of methods. Then we started using the fields and stopped using the method. I would suggest opening a separate issue if this is the case as this one was related to adding the field resolvers feature, which is already done.Originally posted by @pavelnikolov in #282 (comment)
The text was updated successfully, but these errors were encountered: