Skip to content

develop -> main#633

Merged
dh2906 merged 1 commit into
mainfrom
develop
May 22, 2026
Merged

develop -> main#633
dh2906 merged 1 commit into
mainfrom
develop

Conversation

@dh2906
Copy link
Copy Markdown
Contributor

@dh2906 dh2906 commented May 22, 2026

🔍 개요


🚀 주요 변경 내용


💬 참고 사항


✅ Checklist (완료 조건)

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

* feat: 동아리 상세에 대학 요약 정보 추가

- 웹사이트 동아리 상세에서 대학 이미지와 대학 전체 동아리 수를 함께 내려주도록 응답 계약을 확장

- 현재 동아리 회원 수와 대학 단위 동아리 수를 분리해 클라이언트가 각각의 의미를 명확히 사용할 수 있도록 유지

- 상세 API 통합 테스트로 대학 이미지 URL과 동아리 수 응답을 검증

* fix: 동아리 상세 대학 수 집계 조건 정렬

- 대학별 동아리 수 계산이 웹사이트 동아리 조회 조건과 같은 기준을 재사용하도록 정리

- 추후 노출 조건이 공통 조건에 추가될 때 상세 응답의 university.clubCount만 다른 기준으로 계산되는 일을 방지

- 리뷰에서 지적된 집계 기준 드리프트 가능성을 최소 변경으로 해소
@dh2906 dh2906 self-assigned this May 22, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

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.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 46f9ef28-158e-4602-a529-32ef18d2e84a

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

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.

@dh2906 dh2906 merged commit cb5f450 into main May 22, 2026
7 checks passed
@github-actions
Copy link
Copy Markdown

🧪 JaCoCo Coverage Report (Changed Files)

Summary

  • Overall Coverage: 97.2% ✅
  • Covered Lines: 137 / 141
  • Changed Files: 3

Coverage by File

Class Coverage Lines Status
WebsiteQueryRepository
gg.agit.konect.domain.website.repository
96.3% 104/108
WebsiteService
gg.agit.konect.domain.website.service
100.0% 33/33

📊 View Workflow Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant