- Issue created by @admirlju
- @admirlju opened merge request.
- Status changed to Needs review
over 1 year ago 11:55am 13 September 2023 - 🇸🇮Slovenia deaom
I do agree for the order not being important for the searchable and filterable attributes, but for sortable it's important that they keep the order in which they are set in the search API fields, as if I'm not mistaken, the first one added (on top) is the one with more "importance" when sorting, but I could be wrong, so leaving this to needs review.
Looking at the meilisearch documentation. The order of sorting attributes is not important when setting them up, this setting only sets what values can be used for sorting. The sorting order is done at search time, so values in the
"sort"
parameter must be correctly ordered.- Status changed to RTBC
over 1 year ago 8:22am 14 September 2023 - 🇸🇮Slovenia deaom
Thank you for looking into it. Did test the code and it's working as expected and I see no issue with this being sorted before comparison is made, marking it as RTBC.
-
bcizej →
committed e130b973 on 1.x authored by
admirlju →
Issue #3386972 by admirlju, DeaOm: Serchable, sortable and filterable...
-
bcizej →
committed e130b973 on 1.x authored by
admirlju →
- Status changed to Downport
about 1 year ago 3:14pm 21 September 2023 - @bcizej opened merge request.
-
bcizej →
committed e4332d5d on 2.x authored by
admirlju →
Issue #3386972 by admirlju, DeaOm: Serchable, sortable and filterable...
-
bcizej →
committed e4332d5d on 2.x authored by
admirlju →
- Status changed to Fixed
about 1 year ago 3:45pm 21 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.