Skip to content

Dev 인기 급상승 및 오늘의 핫이슈 로직 개선#129

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

Dev 인기 급상승 및 오늘의 핫이슈 로직 개선#129
Qbooo merged 2 commits into
mainfrom
dev

Conversation

@Qbooo
Copy link
Copy Markdown
Contributor

@Qbooo Qbooo commented Apr 9, 2026

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 반응성 점수 업데이트 로직을 최적화하여 점수가 변경되지 않은 경우 불필요한 타임스탬프 갱신이 발생하지 않도록 개선했습니다.

@Qbooo Qbooo merged commit 936c7b7 into main Apr 9, 2026
1 of 2 checks passed
@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: 826b329f-e372-4da0-8619-747bfbdb32c5

📥 Commits

Reviewing files that changed from the base of the PR and between e8470f5 and 582e8c2.

📒 Files selected for processing (1)
  • src/main/java/com/valanse/valanse/domain/Vote.java

개요

Vote.java 파일의 updateReactivityScore() 메서드 로직이 최적화되었습니다. 새로운 점수를 별도로 계산한 후, 현재 값과 다를 때만 reactivityScorereactivityUpdatedAt 필드를 업데이트하도록 변경되었습니다.

변경 사항

응집도 / 파일 요약
반응성 점수 업데이트 최적화
src/main/java/com/valanse/valanse/domain/Vote.java
updateReactivityScore() 메서드에서 불필요한 중복 업데이트를 제거. 점수 값이 변경될 때만 필드를 갱신하도록 개선.

코드 리뷰 예상 소요 시간

🎯 2 (Simple) | ⏱️ ~10 minutes

🐰 한 번의 호출마다 모두 갱신하던 방식에서,
변화만 담아내는 현명함이 피어났네요.
불필요한 업데이트는 멀리하고,
진정한 변화만을 기록하는 깔끔한 로직이요!
효율의 미소로 가득하네 ✨

✨ 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.

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