- First commit to issue fork.
- Status changed to Needs review
almost 2 years ago 2:00am 16 February 2023 - achap π¦πΊ
We also need this. Locally it's working great for me. Merged in latest changes, fixed the failing tests and also fallback to decimal type. Anything else need to be done?
- achap π¦πΊ
A question I have is that there is also a processor for "Number field-based boosting" per item. So maybe it would be good to have separate fields in OS. Right now in OS there is just one
search_api_boost
. I guess you could combine the values from Drupal into one boost field or have separate OS fields. - achap π¦πΊ
Actually. It seems that by the time
$item->getBoost()
is called with both processors enabled the scores get combined. So this already works correctly for both use cases. - Status changed to Needs work
almost 2 years ago 2:04am 20 February 2023 - π¦πΊAustralia kim.pepper πββοΈπ¦πΊSydney, Australia
There seem to be a lot of changes in the MR that don't relate to this issue. π€
- achap π¦πΊ
Hmm that's weird. Do you mind giving an example? When I look at the diff I only see changes related to this issue.
https://git.drupalcode.org/project/search_api_opensearch/-/merge_requests/10/diffs
- Status changed to Fixed
almost 2 years ago 1:49am 21 February 2023 - π¦πΊAustralia kim.pepper πββοΈπ¦πΊSydney, Australia
Oh must have been a weird Gitlab glitch. Seems fine now.
Committed to 2.x. Thanks!
-
kim.pepper β
committed fa23b3e5 on 2.x authored by
larowlan β
Issue #3291381 by achap, larowlan, kim.pepper: Add support for per-type...
-
kim.pepper β
committed fa23b3e5 on 2.x authored by
larowlan β