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: 실시간 채팅 - 기본 메세지 송수신, 세션 처리 #164

Open
wants to merge 43 commits into
base: develop
Choose a base branch
from

Conversation

zzo3ozz
Copy link
Member

@zzo3ozz zzo3ozz commented Jun 1, 2024

무엇을 위한 PR인가요?(: 뒤 설명추가)

  • 신규 기능 추가 : 실시간 채팅 - 기본 메세지 송수신 및 세션 처리
  • 버그 수정 :
  • 리펙토링 :
  • 문서 업데이트 :
  • 기타 :

작업 내역

  • 문서의 양이 PR 페이지에 담기에 길어 Notion 페이지로 대체합니다.
    FamilyMoments Chat Service 개요 문서에서 실시간 채팅 서비스에 대한 기본적인 개요 및 개발 내용을 확인하실 수 있고,
    Spring 실시간 채팅 개발 과정 문서에서 WebSocket, STOMP 등의 보충 설명을 확인하실 수 있습니다.

    🏷️ FamilyMoments Chat Service 개요

    🏷️ Spring 실시간 채팅 개발 과정

PR 특이 사항

  • 추가로 개발 완료된 사항이 있으나, 코드의 양을 고려하여 분리하여 PR 진행하려고 합니다.
    현재 PR 병합 후 추가 PR 작성하도록 하겠습니다!
  • 현재 PR에서 ignore 상태인 파일이 2개 존재합니다. 두 파일 모두 구글 드라이브에 저장해두었습니다.
    PR 병합 이후 각 파일을 아래 패키지에 추가 부탁드립니다!
    • ChatRedisPrefix.java - com.spring.familymoments.domain.chat 내부
    • StompWebSocketConfig.java - com.spring.familymoments.config.secret 내부

zzo3ozz added 30 commits January 8, 2024 01:00
GETTER가 누락되어 builder 패턴이 원하는 대로 실행되지 않음
@zzo3ozz zzo3ozz self-assigned this Jun 1, 2024
Copy link
Contributor

@Hajin-Lee0406 Hajin-Lee0406 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~~! 주석 많아서 읽기 좋아요!!

@NotNull
private String message;
@NotNull
private LocalDateTime sendedTime;
Copy link
Contributor

Choose a reason for hiding this comment

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

response도 validtion 사용하신 이유 궁금합니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

response라는 생각을 안하고 관성적으로 작성한듯 합니다!! 수정하겠습니다!!!!

@@ -5,23 +5,39 @@
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import java.util.Set;
import java.util.concurrent.TimeUnit;

@Service
@Transactional(readOnly = true)
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 의도된 부분인가요??

Copy link
Member Author

Choose a reason for hiding this comment

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

해당 부분 제가 작성한 부분이 아니라서 잘 모르겠습니다만, 제 생각에는 불필요한 것 같긴 합니다!
@spacewalk00 융 확인해주실 수 있으실까요?!

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.

2 participants