@usereactify/search 4.2.2 → 4.2.3

package/CHANGELOG.md

@@ -2,6 +2,13 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [4.2.3](https://bitbucket.org/usereactify/reactify-search-ui/compare/v4.2.2...v4.2.3) (2022-06-23)
+
+
+### Bug Fixes
+
+* include sorting by collection position when using global curation ([141c463](https://bitbucket.org/usereactify/reactify-search-ui/commit/141c46351711c3cff65275e47fdf8e9ed64b0ad8))
+
 ### [4.2.2](https://bitbucket.org/usereactify/reactify-search-ui/compare/v4.2.1...v4.2.2) (2022-06-21)
 
 

package/dist/sensor/SensorSort.js

@@ -70,7 +70,10 @@ const buildSort = (args) => {
     }
     const sorts = [];
     // show pins first
-    if (!globalCuration)
+    if (globalCuration && collection) {
+        sorts.push(mapCollectionPositionSortClause(collection));
+    }
+    else {
         sorts.push({
             "curations.position": {
                 unmapped_type: "long",
@@ -89,6 +92,7 @@ const buildSort = (args) => {
                 },
             },
         });
+    }
     if (0 < curation.boosting.groupings.length) {
         const groupings = curation.boosting.groupings.sort((a, b) => a.position > b.position ? 1 : -1);
         for (const grouping of groupings) {
@@ -115,16 +119,15 @@ const buildSort = (args) => {
             }
         }
     }
-    // finally, sort by elastic score or index order
-    // using index order ensures a consistent output order for queries which don't really
-    // use a relevance score like collections
-    if ("collection" === curation.type)
+    // finally, for collections, if no other sorting is defined, sort by index order to provide
+    // a consistent order
+    if ("collection" === curation.type && sorts.length === 0) {
         sorts.push("_doc");
-    else
+    }
+    // finally, for search, sort by score
+    if ("search" === curation.type) {
         sorts.push("_score");
-    // @todo ideally, once pins are extinguished and there are no boost rules, the rest of the
-    // result should be ordered by the shopify collection position rather than the index order,
-    // but this will need some proper testing to ensure it does not break curations
+    }
     return sorts;
 };
 /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@usereactify/search",
   "description": "React UI library for Reactify Search",
-  "version": "4.2.2",
+  "version": "4.2.3",
   "license": "MIT",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",