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

채팅 관련 객체 KDoc 작성 #103

Merged
merged 2 commits into from
Dec 9, 2023
Merged

채팅 관련 객체 KDoc 작성 #103

merged 2 commits into from
Dec 9, 2023

Conversation

YangSiJun528
Copy link
Collaborator

개요

채팅 관련 객체에 KDoc 작성/보완 하였습니다.
약간의 코드 레이아웃 변경이 존재합니다.

기능적인 수정은 없으므로 크게 중요하지 않게 보셔도 될 것 같습니다.

Copy link
Member

@SungGil-5125 SungGil-5125 left a comment

Choose a reason for hiding this comment

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

개인적으로 읽으면서 코드의 설명 보다는 코드의 사용법 으로 전반적으로 작성이 되어 있다고 생각이 드는데, 요건 노리신걸까요?

@YangSiJun528
Copy link
Collaborator Author

개인적으로 읽으면서 코드의 설명 보다는 코드의 사용법 으로 전반적으로 작성이 되어 있다고 생각이 드는데, 요건 노리신걸까요?

네 일부 기능은 처음 보았을 때, 이해하기 여려울 수 있다고 생각해서 사용법을 설명하는데 집중하긴 했어요.

혹시 기능에 대한 설명이 부족한 부분이 있나요?

@SungGil-5125
Copy link
Member

네 일부 기능은 처음 보았을 때, 이해하기 여려울 수 있다고 생각해서 사용법을 설명하는데 집중하긴 했어요.

혹시 기능에 대한 설명이 부족한 부분이 있나요?

잘 써주신거 같습니다! 조금 더 풀어서 설명해 줄 부분 코멘트 하나 남겼는데, 요거 빼고는 이견 없습니다 🙇

Copy link
Member

@SungGil-5125 SungGil-5125 left a comment

Choose a reason for hiding this comment

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

수고 하셨습니다 👍

@YangSiJun528 YangSiJun528 merged commit 092511f into main Dec 9, 2023
1 check passed
@YangSiJun528 YangSiJun528 deleted the refactor/chat-kdoc branch December 9, 2023 12:06
YangSiJun528 added a commit that referenced this pull request Dec 9, 2023
* 📝 KDoc 작성 및 코드 레이아웃 변경

* 📝 KDoc 보완
YangSiJun528 added a commit that referenced this pull request Dec 10, 2023
* 채팅 기능 리팩토링 (#100)

* 🐛 필드 이름과 타입이 잘못 지정된 문제

* ♻️ UserMessageInfo에 Header 연관관계 추가 및 기존 MessageCustomRepository 추가

UserMessageInfo에 Header 연관관계 추가
UserMessageInfoRepository의 쿼리 기능을 MessageCustomRepository로 이동

* 🔥 사용되지 않는 클래스 제거

* 🐛 StompSender 에러 메시지 반환 시 메시지 타입 잘못 지정되는 문제

* 채팅 관련 객체 KDoc 작성 (#103)

* 📝 KDoc 작성 및 코드 레이아웃 변경

* 📝 KDoc 보완
YangSiJun528 added a commit that referenced this pull request Dec 10, 2023
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