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

Adjust scalar value coersion methods to support the various int types. #23

Closed
wants to merge 0 commits into from

Conversation

bbuck
Copy link
Collaborator

@bbuck bbuck commented Oct 15, 2015

This should resolve #22

@bbuck bbuck closed this Nov 5, 2015
@sogko
Copy link
Member

sogko commented Nov 6, 2015

Hi @bbuck

Sorry for not been giving attention to this earlier. I think issue #22 and #23 is still valid.
Would re-look into this now that the codebase is relatively more stable.

Cheers!

@bbuck
Copy link
Collaborator Author

bbuck commented Nov 6, 2015

I'm currently reimplementing my fix for #22 in the new changes.
On Thu, Nov 5, 2015 at 23:16 Hafiz Ismail notifications@github.com wrote:

Hi @bbuck https://github.com/bbuck

Sorry for not been giving attention to this earlier. I think issue #22
#22 and #23
#23 is still valid.
Would re-look into this now that the codebase is relatively more stable.

Cheers!


Reply to this email directly or view it on GitHub
#23 (comment).

@sogko
Copy link
Member

sogko commented Nov 6, 2015

Awesome, looking forward to that 👍🏻

Thanks again!

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.

Returning a uint in a Resolve function for a GraphQLNonNull(GraphQLInt) field responds with 0
2 participants