Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: popularity isHot 연동 #102

Merged
merged 4 commits into from
Mar 22, 2024
Merged

feat: popularity isHot 연동 #102

merged 4 commits into from
Mar 22, 2024

Conversation

annahxxl
Copy link
Contributor

구현 기능

  • 기존 count 관련 필드로 비교하던 로직을 popularity를 통해 비교하도록 수정했습니다.

resolve: #101

@annahxxl annahxxl added the ✨ Feature 기능 개발 label Mar 21, 2024
@annahxxl annahxxl added this to the 스프린트7 milestone Mar 21, 2024
@annahxxl annahxxl self-assigned this Mar 21, 2024
@annahxxl
Copy link
Contributor Author

annahxxl commented Mar 21, 2024

@KarmaPol 고민되는 부분이 있는데,
entity 단에서 임곗값 환경변수에 바로 접근하여 default값을 넣어줄 수 있도록 구현하길 원했으나,,
entity에서 외부 컴포넌트를 주입할 수 없으니 어쩔 수 없이 isHot() 메서드에 추가적으로 threshold 파라미터를 추가했습니다.
그러다 보니 응답 DTO에서도 계속해서 threshold를 넘겨주는 작업이 필요했습니다.

entity와 도메인 분리가 필요하다고 느껴졌던 작업입니다,, 혹시나 현재 구조에서 해당 로직을 반영할 더 좋은 아이디어가 있다면 의견 주세요!😀

@annahxxl annahxxl requested a review from KarmaPol March 21, 2024 10:26
@KarmaPol KarmaPol merged commit 5befd77 into dev Mar 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

popularity isHot 연동
2 participants