-
Notifications
You must be signed in to change notification settings - Fork 0
[Feat] 채팅 시스템 WebSocket 및 Redis Pub/Sub 기본 구조 구현 #2
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
base: develop
Are you sure you want to change the base?
Conversation
kjunh972
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다
|
|
||
| @SpringBootApplication | ||
| @EnableJpaAuditing | ||
| // 엔티티 , 리포지토리 스캔 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 추가하신 이유가 있을까요? 스캔 안해도 작동할텐데여
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
서비스에서 리포지토리 주입할 때 리포지토리가 다른 모듈에 있어 실행이 안되던데 멀티모듈이 처음이라 혹시 다른 방법이 있는 지 궁금합니다!
|
|
||
| import java.util.List; | ||
|
|
||
| //전체 채팅방 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이런 주석은 제거해주세요
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵 알겠습니다
| //채팅방 내부 | ||
| @Repository | ||
| public interface ChatMessageRepository extends JpaRepository<ChatMessage,Long> { | ||
| //채팅방 메시지 조회 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
무슨 주석일까요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
구분용으로 써놓은건데 pr올릴 때 지우겠습니다
| jobs: | ||
| test: | ||
| runs-on: ubuntu-latest | ||
| # Redis 서비스 추가 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 워크플로우에 레디스를 추가하신 이유가 있으신가요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ci 빌드할 때 레디스가 없다해서 추가해봤는데 이 부분 어떻게 처리하면 좋을지 궁금합니다!
[Feat]/{Team/Chat}/{채팅 엔티티,리포지토리+ WebSocket 및 Redis Pub/Sub 기본 구조 구현}
🎯 작업 내용
* RedisMessageListenerContainer 설정 (채팅방 패턴 구독)
* MessageListenerAdapter 설정 (Subscriber 연동)
주요 변경사항
* RedisMessageListenerContainer 설정 (채팅방 패턴 구독)
* MessageListenerAdapter 설정 (Subscriber 연동)
상세 내용
✅ 체크리스트
📋 API 명세서
💬 리뷰 요청사항 (선택)
📚 참고 자료 (선택)