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

채팅방 목록 유저, 채팅방 정렬 오류 수정 #417

Merged
merged 3 commits into from
Dec 12, 2022

Conversation

bumssooooo
Copy link
Member

@bumssooooo bumssooooo commented Dec 9, 2022

PR Type

  • 기능 추가 🔨
  • 버그 수정 🐞
  • 리팩토링 🚧
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트 ⚙️
  • 기타 사소한 수정 🎸
  • 테스트 🔍

Related Issue(작업 내용)

참고 사항

채팅방 목록에서 최근 메세지가 없는 경우 밑으로 내려가게 되는 문제가 있네요..
최근에 생성한 채팅방이더라도 문자를 안보내면 밑에 존재하게되는.. 좋은 해결 방법이 있을까요?

@bumssooooo bumssooooo added fix 🔨 버그 수정 refactor 🏗 코드 리펙토링 범수 S005 SP 4️⃣ labels Dec 9, 2022
@bumssooooo bumssooooo self-assigned this Dec 9, 2022
Copy link
Member

@sominn9 sominn9 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다. 👍 👍

채팅방 목록에서 최근 메세지가 없는 경우 밑으로 내려가게 되는 문제가 있네요..
최근에 생성한 채팅방이더라도 문자를 안보내면 밑에 존재하게되는.. 좋은 해결 방법이 있을까요?

채팅방 DB가 채팅방 생성날짜 등의 데이터를 갖고 있었다면 채팅 메세지가 없을 때는 생성날짜를 기준으로 비교를 하면 되었을 것 같습니다. 지금 당장은 DB 모델을 수정하는 것 이외에는 기발한 방법이 떠오르지 않네요.

Copy link
Collaborator

@Yuhun-Lee Yuhun-Lee left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

국원님께서 첫 Chat을 만드셨던 방법을 사용하면 가능하지 않을까 싶습니다!
nil이라면 아무것도 없는 것이니 맨 위로 올리면 되지 않을까요?

@bumssooooo
Copy link
Member Author

Nil로 오래된 상태라면 새로온 채팅이 있는 채팅방이 우선일 것 같아서요! @juhoon-lee

@bumssooooo bumssooooo force-pushed the fix/chatroom-sorted-error branch from cab0926 to 09956d4 Compare December 12, 2022 10:09
@bumssooooo bumssooooo merged commit b793983 into develop Dec 12, 2022
@bumssooooo bumssooooo deleted the fix/chatroom-sorted-error branch December 12, 2022 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 🔨 버그 수정 refactor 🏗 코드 리펙토링 SP 4️⃣ 범수 S005
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Repository 비동기 호출 API 리팩토링
3 participants