Conversation
Test Results 18 files 18 suites 6s ⏱️ Results for commit 67d1fea. ♻️ This comment has been updated with latest results. |
|
|
||
| sendNewMemberNotification(studyGroup, member); | ||
| log.info("success to join study group"); | ||
| log.info("success to join study group user_id={} ", user.getId()); |
There was a problem hiding this comment.
요런 것도 어떤 그룹에 참여했는지 정보 주면 좋지 않을까요?
There was a problem hiding this comment.
지금 전체적으로 user id만 추가해준 것들이 많네요!!
만약 그룹 수정 log에서 어떤 유저가 무슨 그룹의 정보를 수정했는지 알 수 있으려면 group id도 함께 표시하는 게 좋지 않을까 싶습니다!
저도 사실 좋은 log를 만들어 본 적이 없었어서 잘 모르지만 아래같은 글 찾아보고 무슨 정보를 넣으면 좋을지 고민해볼 것 같습니담
http://bonschicken.tistory.com/144
+) 커밋 메세지 잘쓰삼!! head가 없어요!!
sh0723
left a comment
sh0723
left a comment
There was a problem hiding this comment.
저도 은수 의견에 동의합니다 !!
상당히 노가다성 작업인데 고생 많으셨어요 ,, 은수 PR만 반영해주시면 될거같네요 !
링크 달아주신 글 잘 읽어봤습니다 log 다는 것도 디테일이 필요하네요 저도 처음엔 group id 를 함께 적어볼까 생각했는데 그리고 저희가 공유하는 로그를 쓰는 규칙이 없으니 이러다 같은 이슈가 반복될 것 같아 이번 코어 타임 때 +) 커밋 메세지 head 어떻게 쓰는 지 잘 모르겠습니다.. ++) 로그 규칙을 정하고 진행하는게 좋을 것 같아서 좀만 미뤄보겠습니다.. |
로그 컨벤션 정하는 것 좋습니다!! 어떤 로그 컨벤션이 좋은지 같이 이야기 해보는 게 좋을 것 같네용
https://velog.io/@shin6403/Git-git-%EC%BB%A4%EB%B0%8B-%EC%BB%A8%EB%B2%A4%EC%85%98-%EC%84%A4%EC%A0%95%ED%95%98%EA%B8%B0 |
|
로그는 코타때 서버끼리 논의해봐요~ 리뷰는 그 때 합니다. |
| deleteAllAboutGroup(group); | ||
|
|
||
| log.info("success to delete study group"); | ||
| log.info("success to delete study group user_id={}, group_id={}", user.getId(), groupId); |
There was a problem hiding this comment.
이런건 target user가 더 중요할까요 아님 탈퇴 시킨 user id가 더 중요할까요?
저는 target user도 들가면 좋긴 할 것 같은데 나중에 하시죠
hwangjokim
left a comment
hwangjokim
left a comment
There was a problem hiding this comment.
다른분 믿고 그냥 어프로브 누릅니다~ (안봄)
* group 수정 * notice log * problem log 수정 * solution log 수정 * user log 수정 * trivial:user,problem log 수정 * trivial : noti log 수정 * trivial:notice log 수정 * trivial:group log 수정 * trivial:merge develop branch
4 participants
📌 Related Issue
close #332
🚀 Description
📢 Review Point