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

Full user search api endpoint #1257

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

kutovoys
Copy link
Contributor

Добавлен api эндпоинт для поиска по людям.
Нейминг эндпоинта - обсуждаем
Поля для вывода - тоже
Дискусия #1256

Я не настоящий сварщик. Прошу ревью)

@kutovoys kutovoys requested a review from vas3k as a code owner September 22, 2024 00:11
@vas3k
Copy link
Owner

vas3k commented Sep 22, 2024

Уже есть же поиск по людям, нахера нам второй?

@vas3k
Copy link
Owner

vas3k commented Sep 22, 2024

Просто добавь параметр в уже имеющийся и все

@kutovoys
Copy link
Contributor Author

По людям есть только для suggest'a
И речь именно про апи.
На данный момент в search только api_search_tags и api_search_users который для suggest'a
Ты предлагаешь добавить в api_search_users ?

@vas3k
Copy link
Owner

vas3k commented Sep 22, 2024

Ну название же говорит само за себя. Изначально эти эндпоинты были для поиска, саджест — это лишь корнер кейс, который был нужен, потому его сделали первым

@kutovoys
Copy link
Contributor Author

Понял, принял.
Переделаю.

@vas3k
Copy link
Owner

vas3k commented Sep 22, 2024

Выбора полей тоже не надо. Это никакой реальной задачи не решает, только добавляет лишней сложности коду. Возвращай объекты как они есть. Если чего-то не хватает — добавь в модель

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.

2 participants