Skip to content

add film recommendations#14

Merged
LightInTheFire merged 4 commits intodevelopfrom
add-recommendations
Oct 18, 2025
Merged

add film recommendations#14
LightInTheFire merged 4 commits intodevelopfrom
add-recommendations

Conversation

@basementdoor
Copy link
Copy Markdown
Collaborator

@basementdoor basementdoor commented Oct 13, 2025

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

  • поскольку используется эндпоинт /users, но запрос фактически возвращает фильмы, более адекватным показалось создание отдельного сервиса для рекоммендаций (как черновой вариант). Включать в UserServiceImpl точно некорректно, как вариант можно перенести часть логики с поиском фильмов в FilmserviceImpl, и тогда в контроллере добавить сервис фильмов.

@basementdoor basementdoor marked this pull request as draft October 13, 2025 18:32
@basementdoor basementdoor marked this pull request as ready for review October 13, 2025 19:02
@LightInTheFire
Copy link
Copy Markdown
Owner

Отдельный сервис хорошее решение, но нужно склеить данные из бд в нем же, а не внутри FilmRepository

@basementdoor basementdoor marked this pull request as draft October 17, 2025 19:02
@basementdoor basementdoor marked this pull request as ready for review October 17, 2025 19:29
@LightInTheFire LightInTheFire merged commit 8e07d62 into develop Oct 18, 2025
2 checks passed
Alextrusk27 pushed a commit that referenced this pull request Oct 18, 2025
* up to date to changes

* fix conflicts

* rebase changes

* update methods for new code

---------

Co-authored-by: Egor Ilyin <basementdoor@yandex.com>
@LightInTheFire LightInTheFire deleted the add-recommendations branch October 18, 2025 07:50
n20va pushed a commit that referenced this pull request Oct 18, 2025
* up to date to changes

* fix conflicts

* rebase changes

* update methods for new code

---------

Co-authored-by: Egor Ilyin <basementdoor@yandex.com>
LightInTheFire pushed a commit that referenced this pull request Oct 22, 2025
* up to date to changes

* fix conflicts

* rebase changes

* update methods for new code

---------

Co-authored-by: Egor Ilyin <basementdoor@yandex.com>
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