-
Notifications
You must be signed in to change notification settings - Fork 12
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update aggregate documentation for 6.x #239
base: 6.x
Are you sure you want to change the base?
Conversation
Related nodes can also be aggregated within a query by accessing the aggregation fields in the connection. | ||
In these fields, you can **count**, aggregate the **nodes** or **edges** fields. | ||
|
||
The same selections and types as before are available in relationship aggregations. |
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.
The same selections and types as before are available in relationship aggregations. | |
The same selections and types as in xref:#_aggregation_fields[] are available in relationship aggregations. |
what does "as before" refer to? asking because a link or even repeating the info might make it easier for readers
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.
TBH, this is not new documentation, just updating examples in this PR, but I presume it refers to the section "
== Aggregation fields"
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.
(tricked myself with github views)
updated suggestion
just 6.x? or also relevant for 7.x? |
@rsill-neo4j This should also be in 7.x However, this docs are for an upcoming release. They shouldn't be merged until that release is ready |
Co-authored-by: Richard Sill <[email protected]>
This PR includes documentation updates Updated pages: |
This PR updates documentation on 6.x to match the changes made on neo4j/graphql#5945