Skip to content

fix: 웹 동아리 등록 URI 수정#664

Merged
JanooGwan merged 1 commit into
developfrom
fix/konect-club-registration-uris
Jun 5, 2026
Merged

fix: 웹 동아리 등록 URI 수정#664
JanooGwan merged 1 commit into
developfrom
fix/konect-club-registration-uris

Conversation

@JanooGwan
Copy link
Copy Markdown
Contributor

@JanooGwan JanooGwan commented Jun 5, 2026

🔍 개요

  • 웹에서 사용하는 동아리 등록 요청 및 동아리 정보 수정 요청 API 경로 앞에 /konect prefix를 추가했습니다.

🚀 주요 변경 내용

  • ClubRegistrationRequestController의 기본 URI를 /clubs에서 /konect/clubs로 변경했습니다.
  • 동아리 등록 요청 통합 테스트의 호출 경로를 새 URI 기준으로 갱신했습니다.

💬 참고 사항

  • 기존 /clubs/registration-requests, /clubs/{clubId}/information-update-requests는 각각 /konect/clubs/registration-requests, /konect/clubs/{clubId}/information-update-requests로 변경됩니다.

✅ Checklist (완료 조건)

  • 코드 스타일 가이드 준수
  • 테스트 코드 포함됨
  • Reviewers / Assignees / Labels 지정 완료
  • 보안 및 민감 정보 검증 (API 키, 환경 변수, 개인정보 등)

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 5, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

동아리 등록 요청 API의 기본 경로가 /clubs에서 /konect/clubs로 변경됩니다. 컨트롤러의 클래스 레벨 @RequestMapping 어노테이션과 통합 테스트의 8개 엔드포인트 URL이 모두 일괄 업데이트되며, 테스트 검증 로직은 유지됩니다.

Changes

API 라우팅 경로 통합

Layer / File(s) Summary
컨트롤러 기본 경로 업데이트
src/main/java/gg/agit/konect/domain/club/controller/ClubRegistrationRequestController.java
ClubRegistrationRequestController의 @RequestMapping"/clubs"에서 "/konect/clubs"로 변경되어 등록 요청 및 정보 수정 요청 엔드포인트의 기본 경로를 갱신합니다.
통합 테스트 엔드포인트 URL 동기화
src/test/java/gg/agit/konect/integration/domain/club/ClubRegistrationRequestApiTest.java
동아리 등록 테스트 5개(누락된 로그인, 이미지 없음, 필수값 누락, 이미지 초과, 소개 초과)와 정보 수정 테스트 3개(누락된 로그인, 미존재 동아리, 필수값 누락)의 MockMvc 요청 경로를 /konect/clubs/...로 일괄 업데이트하며, 예상 응답 상태코드는 유지됩니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • BCSDLab/KONECT_BACK_END#642: 두 PR 모두 ClubRegistrationRequestController의 라우팅을 수정하며, 특히 POST /clubs/{clubId}/information-update-requests 엔드포인트가 기본 경로 접두사 변경에 함께 영향을 받습니다.
  • BCSDLab/KONECT_BACK_END#636: 클럽 등록 요청 API를 ClubRegistrationRequestController에 추가하는 변경으로, 동일한 컨트롤러의 URL 라우팅 영역에서 직접적인 관련성이 있습니다.

Suggested labels

라우팅, 테스트, API

Poem

🐰 경로를 정리하는 작은 변화,
/konect로 담아내는 통일된 구조,
테스트도 함께 춤을 추네,
단순하고 명확한 업데이트,
일관성을 위한 작은 발걸음! 🎯

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed PR 설명이 변경 사항과 관련이 있으며, 경로 변경 및 테스트 업데이트를 명확히 설명합니다.
Title check ✅ Passed PR 제목 "fix: 웹 동아리 등록 URI 수정"은 변경 사항의 핵심을 명확하게 설명하고 있습니다. ClubRegistrationRequestController의 기본 경로를 "/clubs"에서 "/konect/clubs"로 변경한 내용을 잘 반영하고 있습니다.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/konect-club-registration-uris

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@JanooGwan JanooGwan added the 버그 정상적으로 동작하지 않는 문제 상황 관련 이슈입니다. label Jun 5, 2026
@JanooGwan JanooGwan self-assigned this Jun 5, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 5, 2026

🧪 JaCoCo Coverage Report (Changed Files)

Summary

  • Overall Coverage: 100.0% ✅
  • Covered Lines: 4 / 4
  • Changed Files: 1

Coverage by File

Class Coverage Lines Status
ClubRegistrationRequestController
gg.agit.konect.domain.club.controller
100.0% 4/4

📊 View Workflow Run

@JanooGwan JanooGwan changed the title fix: 동아리 등록 요청 URI에 konect prefix 추가 fix: 웹 동아리 등록 URI 수정 Jun 5, 2026
@JanooGwan JanooGwan merged commit 0167150 into develop Jun 5, 2026
5 checks passed
@JanooGwan JanooGwan deleted the fix/konect-club-registration-uris branch June 5, 2026 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

버그 정상적으로 동작하지 않는 문제 상황 관련 이슈입니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant