Skip to content

Added: Shopify GraphQL variations to verify with graphql-java#155

Open
gurveenbagga wants to merge 7 commits intomainfrom
update-shopify-order-graphql-schema
Open

Added: Shopify GraphQL variations to verify with graphql-java#155
gurveenbagga wants to merge 7 commits intomainfrom
update-shopify-order-graphql-schema

Conversation

@gurveenbagga
Copy link
Copy Markdown
Contributor

@gurveenbagga gurveenbagga commented Jan 6, 2026

No description provided.

@gurveenbagga gurveenbagga changed the title Fix documentation syntax in shopify-order-subset.graphqls Added: Shopify GraphQL variations to verify with graphql-java Jan 6, 2026
Copy link
Copy Markdown
Contributor Author

@gurveenbagga gurveenbagga left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've reviewed the SHOPIFY_QUERY_VARIATIONS.md file and found a few inaccuracies relative to the Shopify Admin API spec:

  1. In the OrdersMinimal query (lines 7-24), while the structure is valid, it's worth noting that Shopify's PageInfo also provides hasPreviousPage and startCursor for backward pagination, although hasNextPage and endCursor are sufficient for forward-only flows.
  2. In Query #10 (OrdersComplexFilter), the orders field does not accept a filter argument of type OrderFilter. In the Shopify Admin API, complex filtering is handled via the query: String argument.

I've verified these points against the latest Shopify Admin API schema.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants