Skip to content

Commit 032628d

Browse files
committed
Meilisearch: require all query terms to be matched
Previously only the first term had to be matched. That default meilisearch behavior makes sense for e.g. some kind of autocomplete to find and select a single result. But for filtering issues it means you can't narrow down results by adding more terms. This is also more consistent with other indexers and GitHub.
1 parent b348424 commit 032628d

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

modules/indexer/issues/meilisearch/meilisearch.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -211,10 +211,11 @@ func (b *Indexer) Search(ctx context.Context, options *internal.SearchOptions) (
211211
skip, limit := indexer_internal.ParsePaginator(options.Paginator, maxTotalHits)
212212

213213
searchRes, err := b.inner.Client.Index(b.inner.VersionedIndexName()).Search(options.Keyword, &meilisearch.SearchRequest{
214-
Filter: query.Statement(),
215-
Limit: int64(limit),
216-
Offset: int64(skip),
217-
Sort: sortBy,
214+
Filter: query.Statement(),
215+
Limit: int64(limit),
216+
Offset: int64(skip),
217+
Sort: sortBy,
218+
MatchingStrategy: "all",
218219
})
219220
if err != nil {
220221
return nil, err

0 commit comments

Comments
 (0)