Skip to content
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.

Sign up for GitHub

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

Jump to bottom

nested sorting also on fields #3

Closed
jexp opened this issue Jan 21, 2019 · 4 comments · Fixed by #103
Closed

nested sorting also on fields #3

jexp opened this issue Jan 21, 2019 · 4 comments · Fixed by #103
Assignees
@Andy2003
Milestone
Release Candidate 1

Comments

@jexp
Copy link
Contributor

jexp commented Jan 21, 2019

Support sorting not just on query fields a but also on other ObjectReferences (Relationships and Nodes), needs to use apoc functions for sorting collections inline.
@jexp jexp added this to the Release Candidate 1 milestone Jun 1, 2019
@jexp
Copy link
Contributor Author

jexp commented Jun 28, 2019 

apoc.coll.sort(people, ["name","^age"]) -> name desc, age asc

apoc.coll.sort

@jexp
Copy link
Contributor Author

jexp commented Aug 2, 2019

need to wrap pattern comprehensions into apoc.coll.sort

apoc.coll.sort([pattern-comprehension],[^name,age])

@jexp jexp assigned Andy2003 and unassigned conker84 Aug 24, 2019
@jexp
Copy link
Contributor Author

jexp commented Sep 24, 2019

it will still work using enums, just in fields

like

Movie {
   actors(orderBy:[name_asc])
}

@jexp
Copy link
Contributor Author

jexp commented Sep 24, 2019

like we do it in the plugin

@Andy2003 Andy2003 mentioned this issue Sep 25, 2019
Open
Andy2003 added a commit to Andy2003/neo4j-graphql-java that referenced this issue Aug 5, 2020
@Andy2003
Nested sorting on fields (GH neo4j-graphql#3)
18a5595 
This commit add the following features:

* sorting on multiple properties
* sorting on fields
* augmentation-tests provides a diff on failure that can be viewed in IntelliJ

Bugfix:

* The formatted field for temporal is now a string rather than an object
@Andy2003 Andy2003 mentioned this issue Aug 5, 2020
Merged
Andy2003 added a commit to Andy2003/neo4j-graphql-java that referenced this issue Aug 6, 2020
@Andy2003
Nested sorting on fields (GH neo4j-graphql#3)
9899853 
This commit add the following features:

* sorting on multiple properties
* sorting on fields
* augmentation-tests provides a diff on failure that can be viewed in IntelliJ

Bugfix:

* The formatted field for temporal is now a string rather than an object
Andy2003 added a commit to Andy2003/neo4j-graphql-java that referenced this issue Aug 7, 2020
@Andy2003
Nested sorting on fields (GH neo4j-graphql#3)
891365c 
This commit add the following features:

* sorting on multiple properties
* sorting on fields
* augmentation-tests provides a diff on failure that can be viewed in IntelliJ

Bugfix:

* The formatted field for temporal is now a string rather than an object
@Andy2003 Andy2003 mentioned this issue Aug 28, 2020
Closed
@jexp jexp closed this as completed in #103 Sep 2, 2020
jexp pushed a commit that referenced this issue Sep 2, 2020
@Andy2003
Nested sorting on fields (#103)
ba08a65 
* Nested sorting on fields (GH #3)

This commit add the following features:

* sorting on multiple properties
* sorting on fields
* augmentation-tests provides a diff on failure that can be viewed in IntelliJ
* The formatted field for temporal is now a string rather than an object
* Add test for #115
* adjustments after review
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Andy2003 Andy2003

Labels
None yet
Projects
None yet
Milestone
Release Candidate 1
Development

Successfully merging a pull request may close this issue.

Nested sorting on fields Andy2003/neo4j-graphql-java
3 participants
@jexp @conker84 @Andy2003