[FEAT] 모임 API HTTP Status Response 수정하기

[LimdaeIl]

📝 Pull Request

📌 PR 종류

해당하는 항목에 체크해주세요.

• 기능 추가 (Feature)
• 버그 수정 (Fix)
• 문서 수정 (Docs)
• 코드 리팩터링 (Refactor)
• 테스트 추가 (Test)
• 기타 변경 (Chore)

---

✨ 변경 내용

모임 API HTTP Status Response 수정합니다.

🔍 관련 이슈

• Closes [FEAT] 모임 API HTTP Status Response 수정하기 #53

🧪 테스트

변경된 기능에 대한 테스트 범위 또는 테스트 결과를 작성해주세요.

• 유닛 테스트 추가 / 수정
• 통합 테스트 검증
• 수동 테스트 완료

---

🚨 확인해야 할 사항 (Checklist)

PR을 제출하기 전에 아래 항목들을 확인해주세요.

• 코드 포매팅 완료
• 불필요한 파일/코드 제거
• 로직 검증 완료
• 프로젝트 빌드 성공
• 린트/정적 분석 통과 (해당 시)

---

🙋 기타 참고 사항

리뷰어가 참고하면 좋을 만한 추가 설명이 있다면 적어주세요.

Summary by CodeRabbit

릴리스 노트

• 개선사항
  • API 응답 구조를 표준화하여 HTTP 상태 코드 처리를 일관되게 개선했습니다.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

ApiResponse.success() 메서드의 시그니처를 success(T data)에서 success(int statusCode, T data)로 변경하고, 이를 호출하는 GroupController 및 GroupImageController의 응답 생성 코드를 업데이트하였습니다.

Changes

Cohort / File(s)	Change Summary
API 응답 유틸리티 업데이트 src/main/java/team/wego/wegobackend/common/response/ApiResponse.java	success() 메서드 시그니처 변경: success(T data) → success(int statusCode, T data). HTTP 상태 코드를 명시적 매개변수로 추가
그룹 컨트롤러 업데이트 src/main/java/team/wego/wegobackend/group/presentation/GroupController.java, src/main/java/team/wego/wegobackend/group/presentation/GroupImageController.java	응답 생성 시 HTTP 상태 코드를 포함하도록 ApiResponse.success() 호출 업데이트 (예: 201, 200 등)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

• 주의 사항:
  • 업데이트된 컨트롤러 호출 시 전달되는 HTTP 상태 코드 값들이 의도한 응답 상황과 정확히 일치하는지 확인 필요
  • 다른 컨트롤러에서 ApiResponse.success() 호출이 누락되지 않았는지 전체 코드베이스 검증 권장

Possibly related issues

• [FEAT] 모임 API HTTP Status Response 수정하기 #53: 모임(group) API의 HTTP 상태 응답 수정을 목표로 하며, 본 PR의 ApiResponse.success 시그니처 변경 및 GroupController/GroupImageController 호출 업데이트와 직접적으로 일치합니다.

Possibly related PRs

• PR #35: 동일한 ApiResponse API와 컨트롤러 호출 지점을 수정하므로 연관되어 있습니다.
• PR #30: ApiResponse.success에 HTTP 상태 코드 수용 기능을 추가하고 컨트롤러를 업데이트하는 동일한 패턴입니다.
• PR #39: ApiResponse 클래스와 그 팩토리 메서드 시그니처를 변경하므로 직접 연관됩니다.

Poem

🐰 상태 코드, 명확하게 전달하고~
성공의 응답, 더욱 정직하게~
그룹들의 API, 이제 진짜 제멋대로다! ✨
작지만 단단한 변화, 한 발 더 나아가~

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/group-api-response

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1b95783 and 5c98107.

📒 Files selected for processing (2)

• src/main/java/team/wego/wegobackend/group/presentation/GroupController.java (2 hunks)
• src/main/java/team/wego/wegobackend/group/presentation/GroupImageController.java (1 hunks)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR updates the Group API endpoints to explicitly include HTTP status codes in the ApiResponse.success() method calls. The changes align the Group-related controllers with the pattern already used in other parts of the codebase (AuthController, ImageController, UserController).

• Updated ApiResponse.success() calls to use the success(int status, T data) method signature
• Applied changes to three endpoints: group creation, group attendance, and group image upload
• Ensures HTTP status codes (201, 200) are explicitly included in API responses

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
GroupController.java	Updated create group and attend group endpoints to include explicit HTTP status codes (201 and 200) in ApiResponse
GroupImageController.java	Updated upload images endpoint to include explicit HTTP status code (201) in ApiResponse

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Missing space after comma. Should be HttpStatus.OK.value(), response to maintain consistent formatting with the rest of the codebase.

Suggested change

	HttpStatus.OK.value(),response));
	HttpStatus.OK.value(), response));