Wrap auto-completed list input values with square brackets if required · graphql/graphiql#587

(11 Kommentare) (8 Reaktionen) (0 zugewiesene Personen)TypeScript (15.473 Stars) (1.785 Forks)batch import

buggood first issuegraphql-language-service

Beschreibung

In Gatsby's connections, you can add a sort input like the following:

allMarkdownRemark(sort: { fields: [frontmatter___title], order: DESC}) {
  edges {
    node {
      id
    }
  }
}

But in graphiql, it doesn't enforce that fields must be an array so people regularly complain sorting is broken as they just add a field in the UI and then when they try to copy that into a project it won't work.

Is there a reason graphiql is like this? It seems to be casting the single values to an array which is why it seems to work.

Contributor Guide

Tech Stack: typescriptreact
Domain: frontenddeveloper experiencetooling
Issue Type: feature
Schwierigkeit: 3
Geschätzte Zeit: half day
Aktivitätsstatus: stale
Klarheit: clear
Voraussetzungen: GraphQLReactGraphiQL codebase
Einsteigerfreundlichkeit: 60
Research-Richtung: Investigate the auto complete logic in GraphiQL's query editor to determine where list input values are being suggested without brackets. Look for the component responsible for rendering or inserting auto completed values. Consider adding logic to wrap single values in an array when the field expects a list type. Check the GraphQL schema introspection to identify list types. Review comments in the issue for prior discussion and potential solutions.