kuring-250 동아리 모델 수정

[boiledeggg]

https://kuring.atlassian.net/browse/KURING-250?atlOrigin=eyJpIjoiZGE1YzIxNzg5ODQxNDUwMzhlYjBlYTU1ZTAyMzM3OGUiLCJwIjoiaiJ9

요약

동아리 도메인 모델과 서버 응답 모델의 데이터가 일치하도록 수정했습니다.

참고: ku-ring/ku-ring-api-docs#4

Summary by CodeRabbit

• 새로운 기능

  • 동아리에 고유 ID, 간단요약 및 상세 설명 필드 추가
  • 모임 장소(건물·호실)와 위도·경도 같은 위치 정보 추가
  • 설명 이미지 목록 및 포스터 이미지 지원
  • 구독 여부 및 구독 수 노출

• 개선사항

  • 동아리 분류(카테고리·소속·세부구분) 체계화
  • 모집 상태(BEFORE/RECRUITING/CLOSED/ALWAYS) 명시 및 모집 기간 관리 개선
  • 데이터 구조 및 문서화 정비

[coderabbitai]

Walkthrough

Club 도메인 모델이 재구성되어 필드 추가/삭제와 새로운 타입(ClubCategory, ClubAffiliation, ClubDivision, ClubLocation, RecruitmentStatus 등) 도입 및 ClubRecruitment 구조 변경이 이루어졌습니다. 공개 API 서명이 확장되고 일부 이전 필드가 제거되었습니다.

Changes

Cohort / File(s)	Summary
Club 도메인 모델 재설계 data/domain/src/main/java/com/ku_stacks/ku_ring/domain/Club.kt	Club 데이터 클래스 필드 대대적 변경: id, summary, category, affiliation, division, description, location, applyQualification, posterImageUrl, descriptionImageUrl, isSubscribed, subscribeCount 등 추가. introduction, applyRestriction, clubRoom, logoUrl 제거. ClubRecruitment 에서 content 제거, recruitmentStatus 추가. 새 enum/데이터클래스 ClubCategory, ClubAffiliation, ClubDivision, ClubLocation, RecruitmentStatus 도입 및 KDoc 업데이트.

Possibly related PRs

• kuring-250 동아리 도메인 객체 추가 #661: 동일한 Club 도메인 모델(Club.kt) 관련 대규모 리팩터/확장—ClubRecruitment 및 enum/타입 변경이 겹칩니다.

시

🐰 깡충깡충 코드밭을 지나,
새 타입들이 모여 둥지를 틀었네.
필드도 정리되고 길도 뚜렷해져,
도메인 밭엔 봄바람이 불고,
작은 당근 하나 축복을 전하네 🥕✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경사항의 주요 내용을 명확하게 반영하고 있습니다. 'kuring-250 동아리 모델 수정'은 Club 데이터 클래스의 구조 재설계와 새로운 도메인 모델 도입이라는 핵심 변경을 정확하게 요약합니다.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ 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 fix/kuring-250_club_domain

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

data/domain/src/main/java/com/ku_stacks/ku_ring/domain/Club.kt (1)

117-122: latitude와 longitude의 독립적인 nullable 처리를 확인해 주세요.

위도/경도는 일반적으로 쌍으로 존재하거나 둘 다 없는 경우가 대부분입니다. 서버 응답 모델에서 하나만 null이고 다른 하나는 값이 있는 케이스가 가능한지 확인해 보시고, 그렇지 않다면 별도의 nullable 타입(예: Pair<Double, Double>? 또는 별도 Coordinates 클래스)으로 묶는 것도 고려해볼 수 있습니다.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

Actionable comments posted: 4

🤖 Fix all issues with AI agents

In `@data/domain/src/main/java/com/ku_stacks/ku_ring/domain/Club.kt`:
- Around line 39-50: KDoc for the ClubCategory enum wrongly states "4가지 동아리
카테고리" while the enum ClubCategory (values: ACADEMIC, CULTURE_ARTS, SOCIAL,
ACTIVITIES, OTHERS) defines 5 categories; update the KDoc comment above enum
class ClubCategory to accurately reflect the number or wording (e.g., "5가지" or
remove the numeric claim) so the documentation matches the enum values.
- Around line 19-36: 클래스 Club의 KDoc에서 누락된 프로퍼티 설명을 추가하세요: data class Club의 프로퍼티
리스트 중 subscribeCount에 대해 다른 필드들처럼 `@property` subscribeCount 문장을 추가하여 구독자 수(또는 동아리
구독 수)를 설명하고 단위(예: 정수)를 명시하세요; KDoc은 클래스 정의 위의 기존 `@property` 블록과 동일한 스타일로 작성하여
일관성을 유지하세요.
- Line 17: KDoc for the Club class's posterImageUrl is incorrect (it still says
"동아리 로고 URL"); update the KDoc on the posterImageUrl property in Club (class
Club, property posterImageUrl) to accurately describe it as the poster image URL
(e.g., "동아리 포스터 이미지 URL (없으면 `null`)") so the comment matches the field name and
intent.
- Around line 76-83: The enum contains duplicate Korean display names:
SANGHUH_LIFE_SCIENCE and LIFE_SCIENCE both map to "생명과학대학" (and KU_CONVERGENCE
vs CONVERGENCE_SCI_TECH are similar); confirm the intended unique Korean labels
from the server/requirements and update the enum constants
(SANGHUH_LIFE_SCIENCE, LIFE_SCIENCE, KU_CONVERGENCE, CONVERGENCE_SCI_TECH) so
each has a distinct koreanName string (e.g., change one to "상허생명과학대학" or
"KU융합과학기술원" as appropriate), and adjust any reverse-lookup or UI code that
relies on koreanName to use the corrected values.

[mwy3055]

isRecruiting이 빠지긴 했는데 이건 recruitment로 대체 가능할듯 합니다. 토끼 코멘트만 한번 봐주세요

[mwy3055]

good