Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(relevance): put name, description and eywords on same level #188

Merged
merged 2 commits into from
Apr 28, 2018
Merged

feat(relevance): put name, description and eywords on same level #188

merged 2 commits into from
Apr 28, 2018

Commits on Apr 26, 2018

  1. feat(relevance): put name, description and eywords on same level 

    This will give significantly more relevant results (you can try it out with the index `npm-search-relevance-test` I created to test out these settings.

I had to remove the settings to disable exact and prefix from those attributes, because it otherwise clashes with errors like

```
invalid setting for disablePrefixOnAttributes, you should requests all attributes with same priority: attr1, attr2,  does not exists
```

cc @redox, since you did the original ranking decisions here 😄
    @Haroenv
    Haroenv committed Apr 26, 2018
    Configuration menu
    Copy the full SHA
    3db9a5c View commit details
    Browse the repository at this point in the history

  2. Merge branch 'master' into feat/identical-relevance

    @Haroenv
    Haroenv committed Apr 26, 2018
    Configuration menu
    Copy the full SHA
    52936a0 View commit details
    Browse the repository at this point in the history