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

♻️ order comment list by newest first #447

Open
wants to merge 2 commits into
base: be/dev
Choose a base branch
from

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 7, 2025

댓글 조회 시 예전 댓글부터 보여서 이를 반대 순서로 변경합니다.
무려 id 말고 createdAt 순서로 보여줍니다. 대박

closes #446

Copy link
Contributor Author

Overall Project 89.99% 🍏
Files changed 100% 🍏

File Coverage
CommentService.java 96.6% 🍏

Copy link
Contributor

@tackyu tackyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍👍

Copy link

@HaiSeong HaiSeong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정렬 추가된 부분과 테스트 코드 변경 확인했습니다. 수고하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE Backend ♻️ refactor refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants