Help converting a mutation

[dthisner]

Hello,
Love this tool.

I have problems, I want to convert: "mutation {createOneApp(input: {app: {slug: "amazing-app-name"}}) {app_id}}"

This is what I got:

var CreateApp struct {
	Data struct {
		CreateOneApp struct {
			App_id graphql.String
		}
	} `graphql:"createOneApp(app: {slug: $slug})"`
}

The code that is calling it:

	token, err := GetToken(tools.GetUsername(), tools.GetPassword())
	if err != nil {
		return "", err
	}

	httpClient := tools.CreateClient(token)

	variables := map[string]interface{}{
		"slug": graphql.String(appName),
	}

	client2 := graphql.NewClient(tools.GetGraphqlUrl(), httpClient)
	err = client2.Mutate(context.Background(), &models.CreateApp, variables)
	if err != nil {
		return "", fmt.Errorf("problem creating app %w", err)
	}

Thank you :)

I have read the mutation section in the readme, I am beginning to learn GraphQL and still have not wrapped my head around it

I have problem figuring out how I can get the input section into the request

[dmitshur]

One path you can try is to define a struct that corresponds to the input and pass that in. For example:

type CreateOneAppInput struct {
    App struct {
        Slug string `json:"slug"`
    } `json:"app"`
}

var input CreateOneAppInput
input.App.Slug = "amazing-app-name"

err := client2.Mutate(context.Background(), &models.CreateApp, map[string]interface{}{"input": input})
// ...

That's the general approach I've used for mutations when dealing with GitHub's GraphQL API (see input.go in the githubv4 package).

Alternatively, you might be able to make a smaller adjustment to your code. It seems to be missing the "input" layer. Notably, it's present in the GraphQL mutation you mentioned:

mutation {createOneApp(input: {app: {slug: "amazing-app-name"}}) {app_id}}

But not in the call to Mutate:

[...]
} `graphql:"createOneApp(app: {slug: $slug})"`

Try making that:

[...]
} `graphql:"createOneApp(input: {app: {slug: $slug}})"`

There may be more nuances to get right, but perhaps this'll steer your attempt in the right direction.

[dthisner]

Thank you for your help.
I am trying to get away from using the long string, because we have some really long strings and working on converting them into something easier to read and update.

This is what the mutation looks like in GraphQL playground:

mutation {
  createOneApp(input:{
    app: {
      slug:"dennis2231"
    }
  })
{app_id}}

This is what the response looks like:

{
  "data": {
    "createOneApp": {
      "app_id": "app_4icC9nZDQtS9EOsf"
    }
  }
}

This is what I currently have:

var CreateApp2 struct {
	Data struct {
		CreateOneApp struct {
			App_id graphql.String 
		}
	}
}

type CreateOneAppInput struct {
	App struct {
		Slug string `json:"slug"`
	} `json:"app"`
}

func CreateApp(appName string) (string, error) {
	var input CreateOneAppInput
	input.App.Slug = "amazing-app-name"

	token, err := GetToken(tools.GetUsername(), tools.GetPassword())
	if err != nil {
		return "", err
	}

	httpClient := tools.CreateClient(token)
	client := graphql.NewClient(tools.GetGraphqlUrl(), httpClient)

	err = client.Mutate(context.Background(), &CreateApp2, map[string]interface{}{"input": input})
	if err != nil {
		return "", err
	}

	return string(CreateApp2.Data.CreateOneApp.App_id), nil
}

This is the error I get:

	2022/03/28 10:39:01 tools.go:17: non-200 OK status code: 500 Internal Server Error body:
	"{\"errors\":[{\"locations\":[{\"line\":1,\"column\":37}],\
	"message\":\"Cannot query field \\\"data\\\" on type \\\"Mutation\\\".\"},
	{\"locations\":[{\"line\":1,\"column\":10}],\
	"message\":\"Variable \\\"$input\\\"is never used.\"}]}\n"

also tried:

var CreateApp2 struct {
	Data struct {
		CreateOneApp struct {
			App_id graphql.String
		}
	} `graphql:"createOneApp(input: {app: {slug: $slug}})"`
}

With the same response

[dmitshur]

var CreateApp2 struct {
	Data struct {
		CreateOneApp struct {
			App_id graphql.String 
		}
	}
}

The Data layer isn't needed, package graphql handles that here. Based on the GraphQL mutation, I think you want to write just:

var CreateApp2 struct {
	App_ID string
}
[...]
err = client.Mutate(context.Background(), &CreateApp2, map[string]interface{}{"input": input})
if err != nil {
	return "", err
}
return CreateApp2.App_ID, nil

[dthisner]

Tried it, getting following error:

2022/03/28 14:14:38 tools.go:17: non-200 OK status code: 500 Internal Server Error body: "{\"errors\":
[{\"locations\":[{\"line\":1,\"column\":37}],\"
message\":\"Cannot query field \\\"app_id\\\" on type \\\"Mutation\\\".\"},
{\"locations\":[{\"line\":1,\"column\":10}],\"message\":\"Variable \\\"$input\\\" is never used.\"}]}\n"

[dmitshur]

One of the two errors is "Variable $input is never used." Whoops, I definitely dropped it in my last suggestion. Try this:

var m struct {
	CreateOneApp struct {
		App_ID string
	} `graphql:"createOneApp(input: $input)"`
}
[...]
err = client.Mutate(context.Background(), &m, map[string]interface{}{"input": input})
if err != nil {
	return "", err
}
return m.CreateOneApp.App_ID, nil

(That seems to match the structure suggested in https://github.com/shurcooL/graphql#mutations.)

[dthisner]

It worked :D Thank you for your help :)

It probably is. Think the big difference is the Input struct that you gave me :)

type CreateOneAppInput struct {
	App struct {
		Slug string `json:"slug"`
	} `json:"app"`
}

Final Code:

type CreateOneAppInput struct {
	App struct {
		Slug string `json:"slug"`
	} `json:"app"`
}

var m struct {
	CreateOneApp struct {
		App_ID string
	} `graphql:"createOneApp(input: $input)"`
}

func CreateApp(appName string) (string, error) {
	var input CreateOneAppInput
	input.App.Slug = "amazing-app-name"

	token, err := GetToken(tools.GetUsername(), tools.GetPassword())
	if err != nil {
		return "", err
	}

	httpClient := tools.CreateClient(token)
	client := graphql.NewClient(tools.GetGraphqlUrl(), httpClient)

	err = client.Mutate(context.Background(), &m, map[string]interface{}{"input": input})
	if err != nil {
		return "", err
	}

	return string(m.CreateOneApp.App_ID), nil
}