-
Notifications
You must be signed in to change notification settings - Fork 1
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
채팅 관련 객체 KDoc 작성 #103
채팅 관련 객체 KDoc 작성 #103
Conversation
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.
개인적으로 읽으면서 코드의 설명
보다는 코드의 사용법
으로 전반적으로 작성이 되어 있다고 생각이 드는데, 요건 노리신걸까요?
네 일부 기능은 처음 보았을 때, 이해하기 여려울 수 있다고 생각해서 사용법을 설명하는데 집중하긴 했어요. 혹시 기능에 대한 설명이 부족한 부분이 있나요? |
src/main/kotlin/team/themoment/gsmNetworking/common/socket/message/MessageCode.kt
Outdated
Show resolved
Hide resolved
잘 써주신거 같습니다! 조금 더 풀어서 설명해 줄 부분 코멘트 하나 남겼는데, 요거 빼고는 이견 없습니다 🙇 |
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.
수고 하셨습니다 👍
* 📝 KDoc 작성 및 코드 레이아웃 변경 * 📝 KDoc 보완
* 채팅 기능 리팩토링 (#100) * 🐛 필드 이름과 타입이 잘못 지정된 문제 * ♻️ UserMessageInfo에 Header 연관관계 추가 및 기존 MessageCustomRepository 추가 UserMessageInfo에 Header 연관관계 추가 UserMessageInfoRepository의 쿼리 기능을 MessageCustomRepository로 이동 * 🔥 사용되지 않는 클래스 제거 * 🐛 StompSender 에러 메시지 반환 시 메시지 타입 잘못 지정되는 문제 * 채팅 관련 객체 KDoc 작성 (#103) * 📝 KDoc 작성 및 코드 레이아웃 변경 * 📝 KDoc 보완
개요
채팅 관련 객체에 KDoc 작성/보완 하였습니다.
약간의 코드 레이아웃 변경이 존재합니다.
기능적인 수정은 없으므로 크게 중요하지 않게 보셔도 될 것 같습니다.