Skip to content

Conversation

@ayoung-dev
Copy link
Collaborator

📌 과제 설명

👩‍💻 요구 사항과 구현 내용

  • 커뮤니티 게시글 CRUD 컨트롤러
  • 커뮤니티 댓글 CRUD 컨트롤러
  • 게시글, 댓글 컨트롤러 테스트

✅ PR 포인트 & 궁금한 점

컨트롤러 테스트 어렵네요..

@ayoung-dev ayoung-dev self-assigned this Dec 3, 2024
@ayoung-dev ayoung-dev linked an issue Dec 3, 2024 that may be closed by this pull request
2 tasks
Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다.!!

특히 컨트롤러 테스트 고생많이 하셨어요

@Operation(summary = "작성자별 커뮤니티 게시글 조회", description = "작성자별 커뮤니티 게시글 목록을 조회합니다.")
public ApiResponse<Page<CommunityBoardResponseDto>> getByWriterId(
@PathVariable @CurrentUser UUID writerId,
Pageable pageable
Copy link
Collaborator

Choose a reason for hiding this comment

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

@currentuser 없어도 될 것 같아요!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

수정했습니다!

@Tag(name = "Community Comment Command API", description = "커뮤니티 댓글 생성 수정 삭제 API")
@RequiredArgsConstructor
@RequestMapping("/api/community-board/{boardId}")
@RestController
Copy link
Collaborator

Choose a reason for hiding this comment

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

오.. 여기도 Pathvariable이 가능하군요 신기하네요

Copy link
Collaborator

@m-a-king m-a-king left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

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

LGTM!!

@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 4, 2024

@ayoung-dev ayoung-dev merged commit c82a7f5 into main Dec 4, 2024
3 checks passed
@ayoung-dev ayoung-dev deleted the feature/120-add-community-controller branch December 4, 2024 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 커뮤니티 게시글, 댓글 컨트롤러 생성

5 participants