-
Notifications
You must be signed in to change notification settings - Fork 538
Conversation
@IvanGoncharov please review. |
@rybon I have some minor review comments but the main issue here if GraphiQL maintainers will accept this fix or not. |
Yes, I agree. Lets wait and see what they think. Thanks for your feedback! |
Use a lot of ideas from graphql#507 by @rybon
Use a lot of ideas from graphql#507 by @rybon
Use a lot of ideas from graphql#507 by @rybon
Cool! |
@IvanGoncharov following on the discussion in graphql/graphiql#830, would it be possible to extend the options object with a callback field that can be called on every query, so we can make use of the I'm curious what you think. I think I can make a PR that would look very similar to this one based on this approach. |
@rybon I'm not sure about this proposal I need to think about it. |
Use a lot of ideas from graphql#507 by @rybon
Dependent on graphql/graphiql#830
Extends the
graphiql
option to alternatively accept an object instead of a boolean. This object has a field calledautoPrettify
that when set totrue
will pass theautoPrettify
prop to the GraphiQL component, that will in turn run the prettify method when it mounts. This means stripped GraphQL queries will automatically be pretty printed when opened in the GraphiQL editor.