-
Notifications
You must be signed in to change notification settings - Fork 47
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
Resolve Should Accept a Map of String Keys #54
Comments
Should be possible either way. Feel free to rephrase this as a bug and I'll take a look at it. |
@joshprice ok will do. I have dead time tomorrow so i'll work on making a reproduction and will re-edit this as a bug. |
@joshprice I just uploaded a reproduction repo that has a working and notWorking query. These can both be executed with:
and the meat of it is here in
|
Thanks Adam! I've got a fix for this that I should be able to push up later today. |
@joshprice if there's anything I can do to help test just let me know! BTW thanks so much for working on this! I'm working on a GraphQL-RethinkDB example app to show how one can use it with RethinkDB and Phoenix. |
The example app looks great! Is it deployed anywhere? Any chance of linking to it or using as an example in http://playground.graphql-elixir.org/ I didn't manage to get that fix in the last release I'm afraid, still had a couple of things to confirm. Will push something up asap. |
Thanks! I'll deploy it to Heroku and ad to the playground list once I get the mutations finished. |
Awesome will let you know when I've made the fix. Thanks again! On Mon, Feb 15, 2016 at 4:11 AM, Adam Brodzinski notifications@github.com
|
When I return a database document in the resolve function it only works if the map has symbol keys. Currently you have to convert them after every database query or the GraphQL response is
null
for each string key.For example this works:
however this doesn't work
The text was updated successfully, but these errors were encountered: