Api: ✏️ Append LastMessage Field in ChatRoomDetail Response #196
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
작업 이유
ChatRoomDetail
, the server must include thelastMessage
field.unreadMessageCount
. Since any user in the chatroom can send message simultaneously, even if the server retrieves an earlier message, the client will replace thelastMessage
component with the latest message from the the socket server.작업 사항
lastMessage
, this field will benull
{}
, but this work is so hard...lastMessage
field isn't required and is alwaysnull
리뷰어가 중점적으로 확인해야 하는 부분
발견한 이슈