GraphQL Error When Sorting on Node Field

[Undistraction]

Description

Sorting based on field raises a GraphQL Error error

Steps to reproduce

Using http://localhost:8000/__graphql I can run the following query:

{
  allMarkdownRemark {
    edges {
      node {
        fields {
          title
          type
          slug
          date
          author
          category
        }
      }
    }
  }
}

Which shows me all my nodes with all fields populated. However if I try and sort based on any of these fields, for example on date:

{
  allMarkdownRemark(
    sort: { fields: [date], order: DESC }
  ) {
    edges {
      node {
        fields {
          title
          type
          slug
          date
          author
          category
        }
      }
    }
  }
}

Expected result

I would expect the query to return the results sorted by date.

Actual result

GraphQL Error Argument "sort" has invalid value {fields: [date], order: DESC}.
In field "fields": In element #0: Expected type "MarkdownRemarkConnectionSortByFieldsEnum", found date.

Notes

My understanding is that these fields should be added dynamically and that I should be able to sort on them. Trying to filter on any field results in the same error. I am only able to sort on markdown fields. These issues suggest this should be possible #2886 #2024.

Environment

File contents (if changed)

gatsby-config.js: N/A
package.json: N/A
gatsby-node.js: N/A
gatsby-browser.js: N/A
gatsby-ssr.js: N/A

[pieh]

Can you try if this would work?

-sort: { fields: [date], order: DESC }
+sort: { fields: [fields___date], order: DESC }

[Undistraction]

@pieh Thanks. Yes that does work. Is my original approach invalid then? Is this the correct way to query for sub fields?

For anyone stumbling on this, you can go deeper than one level with this approach, for example to sort on node.fields.metadata.keywords:

sort: { fields: [fields___metadata___keywords], order: DESC }

[pieh]

Yeah, it's invalid but it's understandable (double use of fields word) - fields input param is to specify path to field in note that is used for sorting and isn't actually tied to node.field - so if path is node.field.something, we need to use field___something. If path would node.date, then we would use just date.

[Undistraction]

@pieh That makes sense. Thanks for clarifying.