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

Upgrade graphiql version to 0.17.5. #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

DArtagan
Copy link

Updated the values on my machine and tested it out - things all seemed to be running fine.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.548% when pulling cde885c on DArtagan:master into 0137ca1 on graphql-python:master.

@acao
Copy link

acao commented Mar 12, 2020

@DArtagan to add, we now have a min.css as well. Note that difference in bundle size, too! we have an updated cdn example if you're interested.

I would also suggest using the latest react 16 sources, since react 15 hasn't seen a release or bugfix in over two years. with the new major version of GraphiQL in the works (alpha currently), we will switch to requiring 16.4 or later

@KingDarBoja
Copy link
Contributor

This pull request should be aligned with #69 in order to provide offline support for graphiql. @jkimbo

@agates4
Copy link

agates4 commented May 19, 2020

will this be getting merged?

@KingDarBoja @jkimbo

@acao
Copy link

acao commented May 19, 2020

fyi, we should have a pretty plain 1.0.0 stable soon, just some bugfixes left. the alpha has some improvements already.

that said, 0.17.5, and the outstanding bugs are not major blockers, so 0.17.5 might be all you need for now.

here is a demo of our latest 1.0.0-alpha.8 via cdn:
https://graphiql-test.netlify.app/

this is the one that will require react 16.4 or greater.

2.0.0 RC is under development, and should have pretty good API parity.

@ulgens
Copy link

ulgens commented Dec 6, 2021

@jkimbo @syrusakbary Guys, any luck here?

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

Successfully merging this pull request may close these issues.

6 participants