Skip to content

1 commit:#5

Merged
russuAV merged 4 commits intomainfrom
add-database
Aug 11, 2025
Merged

1 commit:#5
russuAV merged 4 commits intomainfrom
add-database

Conversation

@russuAV
Copy link
Copy Markdown
Owner

@russuAV russuAV commented Mar 25, 2025

  • add database schema

- add database schema
Comment thread README.md
Comment thread src/main/java/ru/yandex/practicum/filmorate/model/Film.java
Comment thread src/main/java/ru/yandex/practicum/filmorate/model/filmdata/MpaRating.java Outdated
Comment thread src/main/java/ru/yandex/practicum/filmorate/model/Film.java
- реализована логика запросов и подтверждения дружбы
- доработан функционал для соответствия ТЗ
Comment thread src/main/java/ru/yandex/practicum/filmorate/service/UserService.java Outdated
Comment thread src/main/java/ru/yandex/practicum/filmorate/service/UserService.java Outdated
Comment thread src/main/java/ru/yandex/practicum/filmorate/service/UserService.java Outdated
@russuAV russuAV force-pushed the add-database branch 3 times, most recently from db03c88 to 1421e25 Compare April 7, 2025 20:07
- перенесены данные в БД
- добавлены DAO
- добавлены интеграционные тесты
Copy link
Copy Markdown

@Sla-als Sla-als left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Решение выглядит хорошо структурированным и расширяемым.
Есть небольшое предложение:

Comment thread src/main/java/ru/yandex/practicum/filmorate/controller/LikeController.java Outdated
- функциональность LikeController объединена в одном FilmController'е
Copy link
Copy Markdown

@Sla-als Sla-als left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Отлично!

@russuAV russuAV merged commit c11393a into main Aug 11, 2025
2 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.

3 participants