Rename Server::endpoint to Server::graphql_path
#1606
Assignees
Comments
|
also: see: #1500 for broader concerns. |
garypen
added a commit
that referenced
this issue
Aug 25, 2022
This introduces three new top-level configuration groupings: - Listeners (currently only 1 for data, but will add admin support) - Paths (health, graphql) - GraphqQL (introspection, experimental_defer, experimental_recursion) and retains the existing Server configuration. - Server (cors, sandbox) The purpose of the grouping is to be functional in nature. Clearly, plugins don't fit in with this, but that's one for the future... Various configuration options are extracted from Server into these new groupings. The purpose of the new groups Some renaming is done as well, e.g.: landing_page -> sandbox, endpoint -> graphql The docs are mainly modified, but I would like to re-structure them a bit to reflect this grouping. I'm aware that this goes way beyond the remit of 1606, but I figured I may as well do some tidying up now, since the current situation is far from ideal. fixes: #1606
abernix
added a commit
that referenced
this issue
Aug 25, 2022
This more accurately represents the behavior of the option which changes the path at which your Router will respond to GraphQL requests. The previous name gave the impression that it might be the full "endpoint" which it was not (again, just a path!) Closes #1606
o0Ignition0o
pushed a commit
that referenced
this issue
Aug 26, 2022
This more accurately represents the behavior of the option which changes the path at which your Router will respond to GraphQL requests. The previous name gave the impression that it might be the full "endpoint" which it was not (again, just a path!) Closes #1606 Co-authored-by: Gary Pennington <gary@apollographql.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Today, we have
endpointin ourserverconfiguration of our YAML. Its default value is/. You might set it to something like/my-graphql-endpoint.The team is not convinced that
/or/graphqlor/my/graphql/endpointare actually "endpoint"s without the rest of the scheme, host, port, prefix. In other words, a full URL.Rename this to
graphql_pathsince that more accurately defines what it is.The text was updated successfully, but these errors were encountered: