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

[#123] Refactor: 채팅방 페이징 정렬기준 및 날짜형식 리팩토링 #124

Merged
merged 4 commits into from
Jun 2, 2024

Conversation

tkguswls1106
Copy link
Member

PULL REQUEST

🎋 Branch Name

refactor/#123

🔑 Main Contents

  • '채팅방 무한 스크롤 Slice 페이징'에 대한 정렬기준을 다음으로 설정.
    • 페이징을 할 때, 최신 날짜인 마지막 페이지를 0으로 설정하여 역순으로 페이징.
    • 한 페이지 내의 채팅은 날짜 오름차순으로 정렬.
    • 만약 퇴장한 경우에는, 본인 퇴장시각 이하까지의 채팅 내역을 조회.
    • => 알고리즘 : DESC로 DB에서 페이징 조회후, 각 페이지별로 날짜 오름차순 정렬.
  • 실시간 STOMP 통신 날짜형식 리팩토링
    • LocalDateTime을 쪼개어 배열로 보내던 방식을 기존 자료형 그대로 보내는것으로 수정.
    • => RabbitConfig의 날짜 컨버팅 설정을 true에서 false로 전환하여 해결.

🏞 Screenshots (Optional)

image

📋 Checks for reviewers (Optional)

위의 스크린샷에서 보시다시피, 채팅방 페이징 정렬기준에 대한 수정 리팩토링이 성공적으로 이루어졌습니다.

@tkguswls1106 tkguswls1106 added the ♻️ Refactor 코드 리팩토링 label Jun 2, 2024
@tkguswls1106 tkguswls1106 requested a review from JunSang1121 June 2, 2024 14:04
@tkguswls1106 tkguswls1106 self-assigned this Jun 2, 2024
Copy link
Collaborator

@JunSang1121 JunSang1121 left a comment

Choose a reason for hiding this comment

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

승인합니다. 고생하셨어요!!

@tkguswls1106 tkguswls1106 merged commit 1333adf into develop Jun 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] 채팅방 페이징 정렬기준 및 날짜형식 리팩토링
2 participants