Skip to content

Dev#131

Merged
Qbooo merged 2 commits into
mainfrom
dev
Apr 9, 2026
Merged

Dev#131
Qbooo merged 2 commits into
mainfrom
dev

Conversation

@Qbooo
Copy link
Copy Markdown
Contributor

@Qbooo Qbooo commented Apr 9, 2026

Summary by CodeRabbit

변경 사항

  • 개선 사항
    • 투표 순위 지정 알고리즘을 개선했습니다. "핫 이슈" 및 "트렌딩" 투표 선택 기준이 변경되어 더 정확한 투표 순위를 제공합니다.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 9, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c584915a-751e-420d-b33a-1301f1db3754

📥 Commits

Reviewing files that changed from the base of the PR and between 936c7b7 and e761ce8.

📒 Files selected for processing (2)
  • src/main/java/com/valanse/valanse/repository/VoteRepository.java
  • src/main/java/com/valanse/valanse/service/VoteService/VoteServiceImpl.java

Walkthrough

투표(Vote) 데이터를 조회하는 쿼리 로직이 변경되었습니다. reactivityUpdatedAt 필드 기반 필터링에서 createdAt 필드 기반 필터링으로 전환되었으며, 저장소에 새로운 쿼리 메서드가 추가되고 서비스 계층에서 해당 메서드를 사용하도록 업데이트되었습니다.

Changes

Cohort / File(s) Summary
저장소 쿼리 메서드 추가
src/main/java/com/valanse/valanse/repository/VoteRepository.java
createdAt 필드의 시간 범위를 기준으로 투표를 조회하는 새로운 쿼리 메서드 findTopByCreatedAtBetweenOrderByReactivityScoreDescCreatedAtDesc() 추가. 결과는 reactivityScore 내림차순 및 createdAt 내림차순으로 정렬됨.
서비스 계층 쿼리 로직 변경
src/main/java/com/valanse/valanse/service/VoteService/VoteServiceImpl.java
이전 날짜 및 지난 7일의 "핫 이슈" 및 "트렌딩" 투표 조회 로직에서 reactivityUpdatedAt 필터링을 createdAt 필터링으로 변경. 기존 정렬 순서는 유지됨.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 새로운 쿼리 메서드, 반짝반짝해
createdAt으로 변경되어 깔끔하게
핫 이슈와 트렌딩을 더 잘 찾아내고
시간순 정렬로 최고의 투표를 고르네
코드는 우아해, 리뷰는 수월해! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev

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.

@Qbooo Qbooo merged commit 7ac4343 into main Apr 9, 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.

1 participant