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

Feat/exclude merchants from search #89

Merged
merged 3 commits into from
Dec 24, 2024

Conversation

comfrt1k
Copy link
Contributor

@comfrt1k comfrt1k self-assigned this Dec 20, 2024
@TorinAsakura TorinAsakura requested review from looee1q and removed request for TorinAsakura December 23, 2024 10:04
@comfrt1k comfrt1k requested review from looee1q and removed request for looee1q December 23, 2024 10:07
@looee1q
Copy link
Contributor

looee1q commented Dec 23, 2024

В задании сказано:

Функциональность должна работать для обоих типов поиска: full_search и instant_search.

У тебя функциональность реализована только для intant__search. Нужно добавить и для full_search. Хотя на скрине есть логи и для full_search. Может не хватает коммита?

P.S. Отмена замечания. Я поторопился, все норм.

@comfrt1k comfrt1k requested a review from looee1q December 23, 2024 14:58
@looee1q looee1q requested a review from TorinAsakura December 23, 2024 15:46
@TorinAsakura TorinAsakura merged commit f4d98d4 into master Dec 24, 2024
1 check passed
@TorinAsakura TorinAsakura deleted the feat/exclude-merchants-from-search branch December 24, 2024 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants