Merge pull request #17187 from codermango/fix/by-query-api

fix(CatalogClient): sortField to orderField

.changeset/eight-trainers-smoke.md

@@ -0,0 +1,5 @@
+---
+'@backstage/catalog-client': patch
+---
+
+Fixed bug in `queryEntities` of `CatalogClient` where the `sortField` is supposed to be changed to `orderField`.

packages/catalog-client/src/CatalogClient.test.ts

@@ -352,7 +352,7 @@ describe('CatalogClient', () => {
         ],
       });
       expect(mockedEndpoint.mock.calls[0][0].url.search).toBe(
-        '?limit=100&sortField=metadata.name,asc&sortField=metadata.uid,desc&fields=a,b&fullTextFilterTerm=query',
+        '?limit=100&orderField=metadata.name,asc&orderField=metadata.uid,desc&fields=a,b&fullTextFilterTerm=query',
       );
     });
 

packages/catalog-client/src/CatalogClient.ts

@@ -230,7 +230,7 @@ export class CatalogClient implements CatalogApi {
       }
       if (orderFields !== undefined) {
         (Array.isArray(orderFields) ? orderFields : [orderFields]).forEach(
-          ({ field, order }) => params.push(`sortField=${field},${order}`),
+          ({ field, order }) => params.push(`orderField=${field},${order}`),
         );
       }
       if (fields.length) {