-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add breadcrumb info to GraphQL categories topic #2767
Conversation
guides/v2.3/graphql/index.md
Outdated
|
|
||
|
|
||
| {:.bs-callout .bs-callout-info} | ||
| You can access the GraphQL introspection feature only if your Magento instance is in developer mode. See [Set the Magento mode]({{ page.baseurl }}/config-guide/cli/config-cli-subcommands-mode.html) for more information about modes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider leaving off "for more information about modes" -- just
"See [Set the Magento mode]."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's too terse for me. Changed to
[Set the Magento mode] describes how to check and change the mode.
| `default_sort_by`| String | The attribute to use for sorting | ||
| `products(<attributes>)` | `CategoryProducts` | The list of products assigned to the category | ||
| `children` | `CategoryTree` | | ||
| `breadcrumbs` | `Breadcrumb` | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Breadcrumb description missing because it's obvious, right?
Further down page: Breadcrumb object --- Make Breadcrumb link to the Breadcrumb object
or in description: A Breadcrumb object that contains a set of links to show customers where they are in relation to other ....
| `sort` | `ProductSortInput` Specifies which attribute to sort on, and whether to return the results in ascending or descending order. See [Searches and pagination in GraphQL]({{ page.baseurl }}/graphql/search-pagination.html) for more information. | ||
|
|
||
| #### CategoryProducts | ||
| `sort` | `ProductSortInput` | Specifies which attribute to sort on, and whether to return the results in ascending or descending order. See [Searches and pagination in GraphQL]({{ page.baseurl }}/graphql/search-pagination.html) for more information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider dropping "for more information"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed to [Searches and pagination in GraphQL] describes sort orders.
|
|
||
| The `category` endpoint allows you to search for a single category definition or the entire category tree. | ||
| The `category` endpoint allows you to search for a single category definition or the entire category tree. To return multiple category levels in a single call, define the response so that it contains up to ten nested `children` options. You cannot return the entire category tree if it contains more than 10 sublevels. | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if it helps, but consider adding a "for example"
define a response so that it contains up to ten nested children options, for example the query can return a CategoryTree object with up to ten levels of categories, and each of those categories can contain up to ten CategoryTree objects with up to ten subcategories.
Also, in the Sample Queries section -- It might be good to show a real response for the CategoryTree query since it's kind of complicated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left the description as is, but added sample output.
meker12
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Breadcrumb description lost it's tail ;-)
| ``` | ||
| ``` | ||
| category ( | ||
| id: int |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing the end of the Breadcrumb description.....
| `breadcrumbs` | `Breadcrumb` | A `Breadcrumb` object contains information about each category that | ||
| `children` | `CategoryTree` | A `CategoryTree` object that contains information about a child category. You can specify up to 10 levels of child categories. | ||
|
|
||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the way you changed the "See" reference 🥇
| --- | ||
|
|
||
| The `category` endpoint allows you to search for a single category definition or the entire category tree. | ||
| The `category` endpoint allows you to search for a single category definition or the entire category tree. To return multiple category levels in a single call, define the response so that it contains up to ten nested `children` options. You cannot return the entire category tree if it contains more than 10 sublevels. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a note, nesting level limit can be re-configured via di.xml by the system integrator.
Add support for OpenSearch for on-premises installation
This PR is a:
Summary
When this pull request is merged, it will document the breadcrumb support added in GraphQL PR-112
whatsnew
Added information about breadcrumbs in the GraphQL category topic.