Skip to content

Fix GraphQL example schema construction#428

Draft
funwithcthulhu wants to merge 1 commit into
camlworks:masterfrom
funwithcthulhu:temp/graphql-example-schema
Draft

Fix GraphQL example schema construction#428
funwithcthulhu wants to merge 1 commit into
camlworks:masterfrom
funwithcthulhu:temp/graphql-example-schema

Conversation

@funwithcthulhu
Copy link
Copy Markdown
Contributor

Summary

  • Build the GraphQL object and schema through functions so the example does not keep weak type variables at top level.
  • Apply the same shape to the Reason example and keep both README snippets in sync.

Fixes #373.

Validation

  • git diff --check

Blocked locally:

  • opam exec -- dune build example/i-graphql/graphql.exe needs graphql-lwt, which is not installed in this switch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Error in GraphQL examples

1 participant