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

Fix no schema on GraphiQL #1050

Merged
merged 1 commit into from
Nov 6, 2018
Merged

Fix no schema on GraphiQL #1050

merged 1 commit into from
Nov 6, 2018

Conversation

sabondano
Copy link
Contributor

Resolves #1042

Motivation

  • For the schema to be available at /graphiql

Changelog

Bug Fixes

  • Changing GraphQL max_complexity from 50 to 200 for introspection to
    be possible on GraphiQL.
  • Editing test that relied on a max_complexity of 50.

@sabondano sabondano self-assigned this Nov 5, 2018
@ghost ghost added the in progress label Nov 5, 2018
@coveralls
Copy link

coveralls commented Nov 5, 2018

Pull Request Test Coverage Report for Build b0f1b0a2-a1f5-4f71-a7f3-4ccf45178000

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 89.14%

Totals Coverage Status
Change from base Build 6ba84e67-f08a-4ebd-affa-c9de0b58f0f2: 0.0%
Covered Lines: 3308
Relevant Lines: 3711

💛 - Coveralls

Copy link
Contributor

@zachdaniel zachdaniel left a comment

Choose a reason for hiding this comment

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

Agree with @KronicDeth that a comment would be good. Other than that, looks good :)

Why:

* For the schema to be available at `/graphiql`
* Issue link: #1042

This change addresses the need by:

* Changing GraphQL `max_complexity` from 50 to 200 for introspection to
be possible on GraphiQL.
* Editing test that relied on a `max_complexity` of 50.
@sabondano sabondano added the ready for review This PR is ready for reviews. label Nov 6, 2018
Copy link
Contributor

@acravenho acravenho left a comment

Choose a reason for hiding this comment

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

Works great 👍

@KronicDeth KronicDeth merged commit 593e776 into master Nov 6, 2018
@ghost ghost removed the in progress label Nov 6, 2018
@KronicDeth KronicDeth deleted the sa-graphiql-no-schema-fix branch November 6, 2018 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants