Skip to content
This repository has been archived by the owner on Oct 14, 2020. It is now read-only.

Allow passing variables #13

Closed
wants to merge 1 commit into from
Closed

Allow passing variables #13

wants to merge 1 commit into from

Conversation

dimaMachina
Copy link

I would like to be able to pass variables with query, i made changes in repo, but every time i receive an error:

className: "bad-request"
code: 400
data: {data: null}
errors: [{message: "Unexpected GraphQL argument type "Variable"", line: 3, column: 9, path: "users"}]
message: "GraphQL processing error. (@feathers-plus/graphql)"
name: "BadRequest"
type: "FeathersError"

my service call

return app.service('api/graphql').create({
    query: `
      query($q: String) {
        users(queryBy: $q) {
          _id
          fullName
          email
          manager {
            position
          }
        }
      }
    `,
    variables: {
      q: "dmytro"
    }
  })

@eddyystop
Copy link
Member

Am I correct in thinking that variables are still not supported after applying this PR? If this is the case, I've created #14 referencing this PR.

I'm closing this PR as I don't see its purpose. Feel free to comment if I've missed something.

@eddyystop eddyystop closed this Jan 16, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants