-
Notifications
You must be signed in to change notification settings - Fork 50
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
Cannot get example to work #79
Comments
@neverfox Can you provide an example of some of the queries you are trying? |
Well, I'm a total newbie when it comes to graphQL, but I was trying things from the examples like:
I assume that if was just badly formed, I'd get an error inside of graphiql, but it's just failing at the source code level somehow. Do you know of a query that you know should work that I could try? |
Hm.. I see what you mean.. The GraphiQL instance is not getting access to the database on my end. Tried to spin up the example on my end and no dice either. @neverfox are you able to open up the "Doc" sidebar and see the schema? If not, you probably have the same issue. I'd usually troubleshoot, but I'm strapped for time and I'm not familiar with koa. |
Nope, no schema. |
Make sure that MongoDB is running. |
@tothandras Not sure that's the issue. MongoDB runs as a service on my machine (Ubuntu Wily) and I have no issues using graffiti in my own workflow. |
I’m having the same issue, the first |
I'd love to get this sorted since this is my first experience with considering graphQL for my next project (tired of REST endpoints proliferating like bunnies).
Not only is it running, but I'm looking at the pre-populated data in MongoHub, so I know that was successful. |
Since the example uses Koa, that might be the implicit default body parser there. |
Can you try the examples in the graffiti project? I can't introduce the errors you are facing with. 😕 |
@tothandras Regarding express, the problem arises if you add JSON body parsing as often suggested/needed by third party plugins. The workaround is to not use |
@jbach Would it be also viable to require body parsing by the plugin as other plugins do? :) Just drop it altogether and make it a requirement. |
Probably. I checked up on co-body, it doesn’t seem to prevent double parsing like expressjs/body-parser does. I guess shifting that responsibility to the middleware would be my personal favorite. |
+1 |
Thanks for the report! I will fix this tomorrow. |
@KATT @jbach @kaminskypavel @neverfox @dsifford |
@tothandras , nope, unfortunately still getting also please update the readme to port 8081. |
@tothandras, it does work for me. 👍 |
I did a clean checkout and install, am I missing something? |
Eureka! I was struggling as well and noticed I was using npm Once I did a simple |
what version of node was it? |
|
No matter what I try as a query, I get the following error in devtools (react.min.js):
The text was updated successfully, but these errors were encountered: