Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.
βοΈ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Free
Run ID: 4ee09c49-6826-48d8-a169-0d663ac7324b
You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.
Use the checkbox below for a quick retry:
π Trigger review
Note
π Summarized by CodeRabbit Free
Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.
Comment @coderabbitai help to get the list of available commands and usage tips.
<!-- This is an auto-generated comment: summarize by coderabbit.ai -->
<!-- This is an auto-generated comment: skip review by coderabbit.ai -->
> [!IMPORTANT]
> ## Review skipped
>
> Auto reviews are disabled on base/target branches other than the default branch.
>
>
>
> Please check the settings in the CodeRabbit UI or the `.coderabbit.yaml` file in this repository. To trigger a single review, invoke the `@coderabbitai review` command.
>
> <details>
> <summary>βοΈ Run configuration</summary>
>
> **Configuration used**: Organization UI
>
> **Review profile**: ASSERTIVE
>
> **Plan**: Free
>
> **Run ID**: `4ee09c49-6826-48d8-a169-0d663ac7324b`
>
> </details>
>
> You can disable this status message by setting the `reviews.review_status` to `false` in the CodeRabbit configuration file.
>
> Use the checkbox below for a quick retry:
> - [ ] <!-- {"checkboxId": "e9bb8d72-00e8-4f67-9cb2-caf3b22574fe"} --> π Trigger review
<!-- end of auto-generated comment: skip review by coderabbit.ai -->
<!-- tips_start -->
---
> [!NOTE]
> <details>
> <summary>π Summarized by CodeRabbit Free</summary>
>
> Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting <https://app.coderabbit.ai/login>.
>
> </details>
<sub>Comment `@coderabbitai help` to get the list of available commands and usage tips.</sub>
<!-- tips_end -->
Reviewed the PR changes at 88837d2: no additional code fixes are needed from this comment. I validated branch/workflow status and ran tests locally; current failure is environment-related (tests compiled for Java 21, runner Java 17), not from these controller changes.
> @copilot review
>
Reviewed the PR changes at `88837d2`: no additional code fixes are needed from this comment. I validated branch/workflow status and ran tests locally; current failure is environment-related (tests compiled for Java 21, runner Java 17), not from these controller changes.
devitq
(Migrated from github.com)
reviewed 2026-04-23 19:21:11 +00:00
The author of this PR is on the CodeRabbit Free Plan. In order to use the Chat feature, please upgrade the PR author to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.
<!-- This is an auto-generated reply by CodeRabbit -->
The author of this PR is on the CodeRabbit Free Plan. In order to use the Chat feature, please upgrade the PR author to CodeRabbit Pro by visiting <https://app.coderabbit.ai/login>.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Comment
@coderabbitai helpto get the list of available commands and usage tips.@copilot review
Reviewed the PR changes at
88837d2: no additional code fixes are needed from this comment. I validated branch/workflow status and ran tests locally; current failure is environment-related (tests compiled for Java 21, runner Java 17), not from these controller changes.Fix issues described in the comments below
@@ -8,13 +8,18 @@ import com.project.movienight.application.ports.input.CreateFilmUseCaseimport com.project.movienight.application.ports.input.DeleteFilmUseCaseStar projections for imports, again
@@ -13,4 +17,4 @@import org.springframework.web.bind.annotation.PatchMappingimport org.springframework.web.bind.annotation.PathVariableimport org.springframework.web.bind.annotation.PostMappingimport org.springframework.web.bind.annotation.RequestBodyWe must depend on usecase, not on service
@@ -7,2 +8,4 @@import com.project.movienight.application.ports.input.CreateFilmLibraryCommandimport com.project.movienight.application.ports.input.CreateFilmLibraryUseCaseimport com.project.movienight.application.ports.input.GetAllFilmsUseCaseimport com.project.movienight.application.ports.input.GetFilmByIdUseCaseSame here, do not use star projections for imports
We must depend on usecases, not on service, add usecases as described in previous comments
@@ -8,8 +8,11 @@ import com.project.movienight.application.ports.input.CreateUserUseCaseimport com.project.movienight.application.ports.input.DeleteUserUseCaseDont use star projections (*), use direct imports
Same here, no usecases have been added for those new methods
No use case have been added for those two methods, thus no interface
Add usecases
@coderabbitai summarize
The author of this PR is on the CodeRabbit Free Plan. In order to use the Chat feature, please upgrade the PR author to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.