Skip to content

Conversation

@1117mg
Copy link
Member

@1117mg 1117mg commented Aug 16, 2025

📌 PR 제목

📋 작업 내용

  • feat: 유저의 루트미션 도전/성공/실패 횟수 추가

✅ 체크리스트

  • 테스트를 완료했나요?
  • 코드 컨벤션을 지켰나요?
  • 관련 문서를 업데이트 했나요?

@1117mg 1117mg self-assigned this Aug 16, 2025
@1117mg 1117mg added the enhancement New feature or request label Aug 16, 2025
@1117mg 1117mg linked an issue Aug 16, 2025 that may be closed by this pull request
2 tasks
@claude
Copy link

claude bot commented Aug 16, 2025

Claude finished @1117mg's task —— View job


코드 리뷰 결과

변경 사항 개요:
사용자의 루트미션 도전/성공/실패 횟수를 마이페이지에서 조회할 수 있도록 기능을 추가했습니다. UserStatusResponse에 성공/실패/총도전 횟수 필드를 추가하고, 실제 도전 기록에서 통계를 계산하여 반환하는 로직을 구현했습니다. N+1 문제를 방지하기 위해 JOIN FETCH를 사용한 전용 리포지토리 메서드도 추가했습니다. application.yaml의 포트 변경(8080 → 9090)도 포함되어 있습니다.

리뷰 의견

LGTM

코드가 전반적으로 잘 구현되어 있습니다. N+1 문제를 고려한 JOIN FETCH 쿼리 사용, 스트림을 활용한 깔끔한 통계 계산 로직, 그리고 적절한 null 체크 등이 잘 반영되어 있습니다.

할 일 목록

  • 변경된 파일들 분석
  • 코드 리뷰 및 피드백 작성
  • 최종 리뷰 결과 업데이트

@1117mg 1117mg merged commit 60842ec into main Aug 16, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] 마이페이지 정보조회 API

2 participants