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

[BE] GET /chat/room API 수정 #434

Merged
merged 1 commit into from
Dec 10, 2023
Merged

[BE] GET /chat/room API 수정 #434

merged 1 commit into from
Dec 10, 2023

Conversation

namewhat99
Copy link
Collaborator

이슈

체크리스트

  • 각 채팅방별로 안 읽은 메시지 존재하는지 반환 all_read
  • 전체 채팅방목록에서 안 읽은 메시지 존재하는지 반환 all_read

고민한 내용

  • 각 채팅방별로 안 읽은 메시지 존재여부는 가장 마지막 채팅의 읽음여부만 확인하면 된다

스크린샷

@namewhat99 namewhat99 added review code-review BE BE 관련 chatting 채팅 labels Dec 10, 2023
@namewhat99 namewhat99 self-assigned this Dec 10, 2023
@@ -110,24 +113,36 @@ export class ChatService {
'post.thumbnail as post_thumbnail',
'chat_info.create_date as last_chat_date',
'chat_info.message as last_chat',
'chat_info.is_read as all_read',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

현재 채팅방의 모두 읽음 여부는 마지막 채팅이 읽혔는지만 확인하면 된다

BE/src/chat/chat.service.ts Show resolved Hide resolved
@koomin1227 koomin1227 merged commit a6bf213 into BE Dec 10, 2023
1 check passed
@koomin1227 koomin1227 deleted the BE-fixChatRoom-#433 branch December 10, 2023 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE BE 관련 chatting 채팅 review code-review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants