Skip to content

#626 [Feat] 주간 베스트 관점 조회 API - #627

Merged
HYH0804 merged 2 commits into
devfrom
feat/#626
Nov 8, 2026
Merged

HYH0804 merged 2 commits into
devfrom
feat/#626

Conversation

@HYH0804

@HYH0804 HYH0804 commented Nov 8, 2026

Copy link
Copy Markdown
Contributor

#️⃣ 연관된 이슈

#626

📝 작업 내용

✨ Feat

내용 파일
주간 베스트 관점 조회 API (GET /api/v1/perspectives/weekly-best) PerspectiveController.java, WeeklyBestPerspectiveService.java, WeeklyBestPerspectiveResponse.java
직전 주 좋아요 수 랭킹 쿼리, 사후투표 완료 배틀 조회 PerspectiveLikeRepository.java, WeeklyPerspectiveLikeCount.java, PerspectiveRepository.java, UserBattleRepository.java
토큰 선택 경로 추가 (토큰 없으면 비로그인 통과, 잘못된 토큰은 401) JwtFilter.java, SecurityConfig.java
API 명세, 통합 테스트 perspectives-api.md, WeeklyBestPerspectiveApiIntegrationTest.java

📌 공유 사항

  1. 집계 기간은 직전 완료된 주(월 00:00 ~ 일 24:00, KST)이고, 해당 주에 눌린 좋아요 수로 정렬합니다. 동률은 최신 관점 → id 순입니다.
  2. 대상은 공개 배틀의 PUBLISHED 일반 관점입니다. 숨김, 클래스 관점, 비공개 배틀은 제외합니다.
  3. participated 는 사후투표 완료(COMPLETED) 기준입니다. true 면 해당 댓글, false 면 배틀 진입점으로 앱에서 이동합니다.
  4. 한 번에 최대 25개를 내려주고, 5개씩 페이지 처리는 앱에서 합니다.
  5. 스케줄러나 집계 테이블 없이 조회 시점에 계산합니다.

✅ 체크리스트

  • Reviewer 등록
  • Assignees 등록
  • Type 라벨 등록
  • Development 이슈 연동
  • Merge 브랜치 확인
  • 컨벤션 확인
  • 로컬 실행 확인
  • PR 링크 공유

📸 스크린샷

없음

💬 리뷰 요구사항

  1. JwtFilter 에 추가한 토큰 선택 경로(OPTIONAL_AUTH_PATHS)를 중점적으로 봐주세요.

🤖 Generated with Claude Code

HYH0804 and others added 2 commits November 8, 2026 23:15
- 직전 한 주(월~일, KST)에 눌린 좋아요 수로 일반 배틀 관점 최대 25개 조회
- 사후투표 완료 여부를 participated 로 내려 앱이 이동 경로를 결정
- 토큰 선택 경로를 추가해 비로그인도 조회 가능, 잘못된 토큰은 401 유지

Co-Authored-By: Claude Opus 5.5 <[email protected]>
JdbcTemplate 입력은 hibernate.jdbc.time_zone 변환을 거치지 않아
JVM 시간대가 UTC인 CI에서 집계 기간 경계가 어긋났다.
좋아요 시각도 JPQL로 넣어 조회와 같은 변환을 타게 한다.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
@HYH0804
HYH0804 merged commit ee8b42d into dev Nov 8, 2026
1 check passed
@HYH0804
HYH0804 deleted the feat/#626 branch November 8, 2026 14:28
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