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.

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

Course Search Queries #75

Merged
merged 9 commits into from
Aug 7, 2017
Merged

Course Search Queries #75

merged 9 commits into from
Aug 7, 2017

Conversation

Ostrenkiy
Copy link
Contributor

@Ostrenkiy Ostrenkiy commented Aug 6, 2017

Что произошло

При поиске курсов теперь используется механизм с подсказками для поиска.
Подсказки появляются в следующем порядке:

  1. Текущий запрос пользователя
  2. До трех подсказок, которые пользователь уже искал, отсортированных по следующим параметрам:
    • Сначала те запросы, которые пользователь делал чаще
    • Из запросов с одинаковой встречаемостью выше будет запрос с меньшей длиной
    • Если у запросов длина и встречаемость одинаковы, только бог знает, в каком порядке они будут выдаваться пользователю
  3. Те подсказки, которые пришли с api/search-queries

Конечно, каждая подсказка должна быть уникальной.

Демо

search queries demo 4

@Ostrenkiy Ostrenkiy requested a review from kvld August 6, 2017 14:50
@Ostrenkiy Ostrenkiy self-assigned this Aug 6, 2017
@Ostrenkiy Ostrenkiy added this to the 1.40 milestone Aug 7, 2017
@Ostrenkiy Ostrenkiy merged commit 23fabde into dev Aug 7, 2017
@Ostrenkiy Ostrenkiy deleted the feature/search-queries branch August 8, 2017 17:32
@Ostrenkiy Ostrenkiy mentioned this pull request Aug 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants