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

Support fulltext search API on web #185

Closed
4 of 5 tasks
borovskyav opened this issue Dec 28, 2018 · 7 comments
Closed
4 of 5 tasks

Support fulltext search API on web #185

borovskyav opened this issue Dec 28, 2018 · 7 comments
Assignees
Projects

Comments

@borovskyav
Copy link
Contributor

borovskyav commented Dec 28, 2018

В версии 2.4 появилась возможность использовать полнотекстовый поиск по триггерам, теперь самое время втащить это на фронт

upd 18.01.2019

ToDo:

  • переделать дизайн выдачи триггеров
  • добавить подсветку найденного вхождения
  • добавить кусок описания, если найденное вхождение в нём
  • удаляется поведение по Enter для выбора последнего или единственного найденного тега
  • по Enter теперь отрабатывает поиск
@borovskyav borovskyav created this issue from a note in 2.5 (Ready to develop) Dec 28, 2018
@beevee beevee changed the title Полнотекстовый поиск на фронте Support fulltext search API on web Dec 29, 2018
@sashasushko
Copy link
Contributor

По итогам обсуждения:

  • решить по дизайну
  • обсудить с остальной командой

@sashasushko sashasushko moved this from Ready to develop to Analytics in 2.5 Jan 9, 2019
@borovskyav
Copy link
Contributor Author

borovskyav commented Jan 24, 2019

На бэке: в ответе на метод Search выводить размеченные поля Description и Name, для подсветки на фронте

@borovskyav borovskyav pinned this issue Jan 24, 2019
@titusjaka
Copy link
Contributor

Для подсветки можно использовать стандартный механизм в бливе:
http://blevesearch.com/docs/Highlight%20Matches%20in%20Results/

@kamaev kamaev self-assigned this Feb 4, 2019
@kamaev kamaev moved this from Analytics to Ready to develop in 2.5 Feb 4, 2019
@borovskyav
Copy link
Contributor Author

Предлагаю в результате запроса search (в апи) добавить новое поле highlight с массивом элементов, по размеру равным массиву триггеров, и отсортированным в том же самом порядке.
Поля такие:

TriggerID:
Name:
Description:

Для каждого триггера из массива list предлагаю создавать такой элемент в массиве highlight с тем же самым индексом в массиве и заполнять те поля, в которых наш движок поиска нашел совпадения и подсветил. Например, если при запросе поиска апи нашел совпадения в Desription в триггере SomeId, то объект будет выглядеть так:

TriggerID: SomeID
Description: Что-то из этого будет подсвечено.

@kamaev предлагаю тебе разобраться с тем как bleve умеет подсвечивать совпадения и договориться с @sashasushko о контракте. Если есть какие-то предложения, замечания то давайте обсуждать!

@titusjaka
Copy link
Contributor

Там придётся немного извратиться, потому что сейчас мы из bleve достаём только ID триггеров, отсортированные по внутреннему скору.

@borovskyav
Copy link
Contributor Author

Да, это мы уже поняли. Видимо будем возвращать подобный контракт прямо из индекса.

@borovskyav
Copy link
Contributor Author

Один нюанс @sashasushko не забудь пожалуйста переехать на /search для поиска вместо /page

@borovskyav borovskyav moved this from Ready to develop to In progress in 2.5 Feb 21, 2019
@sashasushko sashasushko moved this from In progress to Ready to develop in 2.5 Mar 22, 2019
@beevee beevee moved this from Ready to develop to In progress in 2.5 Apr 2, 2019
2.5 automation moved this from In progress to Done Apr 12, 2019
@beevee beevee unpinned this issue May 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
2.5
  
Done
Development

No branches or pull requests

4 participants