Mutation struct variable empty? about graphql HOT 3 CLOSED

I receive in the server

As I understand, you have your own GraphQL server where the client is sending the mutation.

I'm not seeing you do anything clearly wrong in the snippet you posted. It looks like it should work.

Are you sure the data in installInfo.Package.Products has non-empty values? Try adding some log statements there to confirm.

The code in graphql package that marshals the variables is very simple:

So it should work. See a demo on the playground: https://play.golang.org/p/LIofG3_Gq1g.

What GraphQL server are you using? Can you dump the raw mutation it receives and see if it looks as expected?

from graphql.

Hi!
I have resolved it by using the json notation, so it passes lowercase variables to the server. I was confused with it because mutation examples do not show how to create the structures for mutation variables.

The solution is to include json:variable

type ProductInput struct {
    Name     graphql.String `json:"name"` 
    Location graphql.String `json:"location"` 
}

Many thanks for you help!

from graphql.

Thanks for following up. I'm glad you got to the bottom of this.

I think I understand what happened now. This general GraphQL client package was factored out of githubql, the GitHub-specific GraphQL client, which I created first. Over there, all the input structs were generated, and they include the json tags. So it's something I completely forgot about that needs to be done in the general case as well.

This is definitely not intuitive, and currently not documented. I should either document this so it's clear this must be done, or improve the graphql code to make it no longer neccessary to manually specify those json tags. They can be inferred from the field name, just need to convert from Go's MixedCaps case to GraphQL's lowerCamelCase (this is already being done for queries, the code for it exists in ident package).

I filed issue 20 for that. Thanks again for the report.

from graphql.