Skip to content

feat: MBTI 팀매칭에 리드·오거나이저 제외 로직 추가#311

Merged
CSE-Shaco merged 7 commits into
GDGoCINHA:developfrom
CSE-Shaco:develop
Mar 11, 2026
Merged

feat: MBTI 팀매칭에 리드·오거나이저 제외 로직 추가#311
CSE-Shaco merged 7 commits into
GDGoCINHA:developfrom
CSE-Shaco:develop

Conversation

@CSE-Shaco

@CSE-Shaco CSE-Shaco commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

📌 연관된 이슈

ex) #이슈번호, #이슈번호

✨ 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

💬 리뷰 요구사항(선택)

Summary by CodeRabbit

릴리즈 노트

  • 신규 기능

    • Rythm8beat 게임 점수 시스템 추가: 점수 등록, 상위 3인 랭킹 조회, 개인 순위 확인 기능
    • 휴대폰 번호 기반 사용자 순위 추적 및 점수 갱신 기능
  • 개선사항

    • MBTI 팀 매칭 알고리즘 강화: 역할 기반 필터링 및 더욱 균형잡힌 팀 배치 로직 적용
    • 크로스 오리진 요청 지원 범위 확대

@coderabbitai

coderabbitai Bot commented Mar 11, 2026

Copy link
Copy Markdown

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

이 PR은 리듬8비트 게임의 점수 제출, 랭킹 조회, 점수 초기화를 위한 REST 엔드포인트를 추가하고, MBTI 팀 매칭 로직을 수정하여 특권 역할 제외 및 혼합 멤버 처리를 지원합니다. CORS 설정도 새로운 출처를 포함하도록 확장됩니다.

Changes

Cohort / File(s) Summary
Rythm8beat 게임 기능
src/main/java/inha/gdgoc/domain/game/controller/Rythm8beatScoreController.java, src/main/java/inha/gdgoc/domain/game/service/Rythm8beatScoreService.java, src/main/java/inha/gdgoc/domain/game/repository/Rythm8beatScoreRepository.java, src/main/java/inha/gdgoc/domain/game/entity/Rythm8beatScore.java, src/main/java/inha/gdgoc/domain/game/dto/request/Rythm8beatScoreRequest.java, src/main/java/inha/gdgoc/domain/game/dto/response/Rythm8beatRanking*.java, src/main/java/inha/gdgoc/domain/game/controller/message/Rythm8beatScoreMessage.java
REST 컨트롤러, 서비스, JPA 저장소, 엔티티 및 DTO를 추가하여 점수 제출, 랭킹 조회 및 초기화 기능을 구현합니다.
MBTI 팀 매칭 개선
src/main/java/inha/gdgoc/domain/admin/game/service/MbtiAdminService.java, src/main/java/inha/gdgoc/domain/admin/game/dto/request/MbtiTeamMatchRequest.java, src/main/java/inha/gdgoc/domain/admin/game/dto/response/MbtiTeamMatchResponse.java, src/main/java/inha/gdgoc/domain/user/repository/UserRepository.java
역할 기반 필터링, 특권 역할(LEAD/ORGANIZER) 제외, MBTI 결과 여부 추적, 혼합 멤버 기반 팀 분산을 지원하도록 MBTI 팀 매칭 로직을 재구성합니다.
데이터베이스 및 보안
src/main/resources/db/migration/V20260309__create_game_scores.sql, src/main/java/inha/gdgoc/global/security/SecurityConfig.java
게임 점수 저장을 위한 테이블 마이그레이션 및 CORS 설정에 새로운 출처 추가.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • PR #308: Rythm8beat 게임 기능 구현과 관련된 동일한 파일들(컨트롤러, 서비스, 저장소, 엔티티, DTO, 마이그레이션, CORS)을 수정합니다.
  • PR #306: MBTI 팀 매칭의 기반을 제공하며, 이 PR에서 MbtiAdminService, MbtiTeamMatchRequest, MbtiTeamMatchResponse를 확장 수정합니다.
  • PR #223: SecurityConfig.java의 CORS 설정을 확장하는 동일한 파일을 수정합니다.

Suggested labels

enhancement

Poem

🐰 새로운 게임이 생겼어, 점수도 올려봐,
팀도 더 공평하게, 역할도 더 잘 살펴봐,
리듬을 타며 랭킹 올라, 우리 함께 더 멀리가,
토끼처럼 빠르고 정확한 매칭, 마법이 일어나!✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PR 제목은 MBTI 팀매칭에서 리드·오거나이저 역할 제외 로직 추가를 명확하게 설명하며, 변경사항의 주요 목적을 정확히 반영합니다.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@CSE-Shaco CSE-Shaco merged commit ea8a6aa into GDGoCINHA:develop Mar 11, 2026
1 of 2 checks passed
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.

2 participants