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

Dev #443

Merged
merged 5 commits into from
Sep 30, 2024
Merged

Dev #443

merged 5 commits into from
Sep 30, 2024

Conversation

sh1nkey
Copy link
Member

@sh1nkey sh1nkey commented Sep 30, 2024

Краткое название

Описание изменений

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

Тестирование

Опишите, как тестировали свои изменения. Например, какие тесты проходят, а какие нет.

Проверка кода

Опишите, как проверить ваш код.

Дополнительная информация

Здесь вы можете добавить какую-либо дополнительную информацию о своих изменениях.

sh1nkey and others added 5 commits September 27, 2024 18:26
optimized chat checking, if it will break
1. `rate-project/details` - роут удален, нигде не использовался

2. Добавлены фильтры аналогично роуту `/projects/` на роут `/rate-project/{program_id}`.
path параметр `program_id` все так же необходим, query параметры в документации.
Актуальным в рамках задачи является query param `is_rated_by_expert`:
`1/true` - только оценённые проекты
`0/false` - только не оцененные проекты
пусто - все проекты
Прочие фильтры могут использоваться по необходимости.

3. Код отрефакторен: логика получения списка изменена, результат аналогичен предыдущему за исключением:
Ключ "scored" добавлен на все проекты (bool).
Ранее был только у оценённых проектов `"scored": true`, сейчас он так же там присутствует, но не оценённые проекты получили ключ `"scored": false`.
…_for_expert

PRO-459: Filter projects for experts
@sh1nkey sh1nkey merged commit 4fe424f into master Sep 30, 2024
2 of 3 checks passed
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