search

chkimes / graphql-net

Convert GraphQL to IQueryable
MIT License
891 stars 86 forks source link

added implementation to resolve enum values #77

Closed epitka closed 7 years ago

epitka commented 7 years ago

see issue #19. Implementation for resolving enum values was missing.

chkimes commented 7 years ago

Good catch - thanks for this!

epitka commented 7 years ago

When will nuget package with this fix be available?

chkimes commented 7 years ago

Version 0.3.5 is available on NuGet with this fix.

chkimes commented 7 years ago

@epitka If you have the time, would you mind adding a test for this fix?

epitka commented 7 years ago

ok, let me see what I can put together...

epitka commented 7 years ago

I created a branch with tests, but when I tried to create pull request I am getting message that I do not have permission to push the changes ?

chkimes commented 7 years ago

Which branch are you trying to push to? It sounds like you might be trying to push to this repo, whereas you should be pushing to your own repo first and then creating a PR. It's either that or you're failing to authenticate with Github when trying to push changes.