Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Return string #493

Closed
wants to merge 6 commits into from
Closed

Return string #493

wants to merge 6 commits into from

Conversation

jamesallison
Copy link

Describe your PR and link to any relevant issues.

I have:

  • Added tests covering the bug / feature (see testing)
  • Updated any relevant documentation (see docs)

@jamesallison jamesallison deleted the return-string branch January 9, 2019 16:31
@@ -7,7 +7,7 @@ import (
"os"
"strings"

"github.com/99designs/gqlgen/codegen"
"github.com/monzo/gqlgen/codegen"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most vendoring tools will let you rewrite the repo without having to change the imports, for dep it you would add this to your Gopkg.toml of your app using gqlgen:

[[constraint]]
  branch = "master"
  name = "github.com/99designs/gqlgen"
  source = "https://github.com/monzo/gqlgen"

@@ -279,6 +281,102 @@ func GraphQL(exec graphql.ExecutableSchema, options ...Option) http.HandlerFunc
return handler.ServeHTTP
}

func TyphonHandlerFromGraphQL(exec graphql.ExecutableSchema, options ...Option) typhon.Service {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than support all transports forever we should work how to make handler open for extension.

The way this is setup currently, everyone using gqlgen would need to fetch monzo/typoon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants