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

Feat : DIG-92 채팅 기능 구현 #24

Merged
merged 3 commits into from
Nov 13, 2023

Conversation

YEJINGO
Copy link
Collaborator

@YEJINGO YEJINGO commented Nov 8, 2023

  • JwtUtils : StompHandler 에서 사용되는 토큰 검증 로직 추가
  • Entity 수정
    • ChatRoom : 채팅방에 필요한 값 추가
    • ChatMessage : 채팅 메세지에 필요한 값 추가
  • build.gradle : websocket,redis 추거
  • ErrorCode : 400으로 되어있는 값들 404 로 변경

Copy link
Collaborator

@bstaran bstaran left a comment

Choose a reason for hiding this comment

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

너무너무 고생 많으셨습니다..! 👍 👍

@bstaran
Copy link
Collaborator

bstaran commented Nov 11, 2023

제가 수정한 부분에서 충돌이 좀 있나봅니다.
미리 죄송합니다. 🥲

@YEJINGO YEJINGO force-pushed the feature/chatting branch 3 times, most recently from b378735 to 13f061b Compare November 13, 2023 01:16
@YEJINGO YEJINGO force-pushed the feature/chatting branch 2 times, most recently from c4c560b to 8d41ede Compare November 13, 2023 02:57
- JwtUtils : StompHandler 에서 사용되는 토큰 검증 로직 추가
- Entity 수정
  - ChatRoom : 채팅방에 필요한 값 추가
  - ChatMessage : 채팅 메세지에 필요한 값 추가
- build.gradle : websocket,redis 추거
- ErrorCode : 400으로 되어있는 값들 404 로 변경
- 좋아요/댓글,대댓글 OAuth2JwtUserDetails 추가
- StompHandler 수정 : CONNECT,SUBSCRIBE,DISCONNET 추가
- ChatMessageService
    - loadMessage 로직 수정 : redis에 저장된 메세지 값에도 인원수에 따라 readCount값을 수정, update된 redis값 추가 설정
    - updateReadCount 추가 : 채팅방에 들어온 인원수에 따라 readCount값을 set 한다. ex) 채팅방에 들어온 인원수가 2일경우 readCount는 0이되고, 그 외의 경우 readCount는 1이된다.
- JwtUtils : token값으로 email 가져오는 로직 추가
@YEJINGO YEJINGO merged commit 48860f1 into Goorm-OGJG:develop Nov 13, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants