Merge pull request #964 from bakaphp/KA-123

refactor: add order
bakaphp · Feb 21, 2024 · bc709b2 · bc709b2
2 parents 28132b2 + ed2f20c
commit bc709b2
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/graphql/schemas/Ecosystem/Admin/app.graphql b/graphql/schemas/Ecosystem/Admin/app.graphql
@@ -141,7 +141,10 @@ extend type Query @guard {
 }
 
 extend type Query {
-    getAppSettings(key: String! @eq): AppSettings
+    getAppSettings(
+        key: String! @eq,
+        orderBy: _@orderBy(columns: ["created_at", "updated_at", "id", "name"])
+    ): AppSettings
         @field(
             resolver: "App\\GraphQL\\Ecosystem\\Queries\\Apps\\AppsListQuery@getAppSettings"
         )

diff --git a/graphql/schemas/Ecosystem/company.graphql b/graphql/schemas/Ecosystem/company.graphql
@@ -183,6 +183,7 @@ extend type Query @guard {
                     "updated_at"
                 ]
             )
+        orderBy: _@orderBy(columns: ["created_at", "updated_at", "id", "name"])
     ): [User!]!
         @paginate(
             builder: "App\\GraphQL\\Ecosystem\\Queries\\Companies\\UserManagementQuery@getAllCompanyUsers"

diff --git a/tests/GraphQL/Ecosystem/Companies/CompanyUserTest.php b/tests/GraphQL/Ecosystem/Companies/CompanyUserTest.php
@@ -63,7 +63,9 @@ public function testGetCompanyUsers()
         $response = $this->graphQL( /** @lang GraphQL */
             '
             {
-                companyUsers() {     
+                companyUsers(
+                    orderBy: [{ field: "id", order: DESC }]
+                ) {     
                     data {
             
                             id,