Skip to content
This repository has been archived by the owner on Nov 16, 2018. It is now read-only.

Initial add of GraphQL. Turns on GraphQL Content and GraphQL Voyager. #61

Open
wants to merge 1 commit into
base: 8.x-1.x
Choose a base branch
from

Conversation

tedbow
Copy link

@tedbow tedbow commented Aug 17, 2017

Add GraphQL support and turn on.

Right now this just turns on GraphQL Content and GraphQL Voyager.

Remaining Tasks:

  1. Determine if we should ship with configuration that presets "Exposed Content" as configured on "admin/config/graphql/content"
  2. Determine which entity types to expose.
  3. Actually test connecting to it.

Remaining questions:

  1. Do we turn on all GraphQL modules?
  2. If we turn on all GraphQL module GraphQL Voyager page become very complicated?
  3. Change the existing toolbar item from "API" to "JSON API" and added new "Graph QL" item that points to the GraphQL Voyager page. is this ok or do we want 1 top level "API" still?
  4. Is the "Authenication" page work for both API modules. If so should be move under "Access Control" toolbar item?

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

Successfully merging this pull request may close these issues.

1 participant