Namo-log / Server

당신의 모든 모임과 함께할, '나모'
https://litt.ly/namong
0 stars 1 forks source link

[Feature] 개인 페이지 모임 기록 #200

Closed hyeonji91 closed 3 days ago

hyeonji91 commented 1 month ago

Type of change

PR Desciption

변경 사항 설명

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요 ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

-

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

개인 페이지 모임 기록 image

그룹 페이지 모임 기록 image

첨부 자료

-

관련 이슈

hyeonji91 commented 1 month ago

[ 1. API 테스트 관련 ] Controller test 추가해보겠습니다

[ 2. 비지니스 Service 분리 관련 ] 서비스 개수가 많아졌다는 측면에서 불편함이 있긴 합니다! 하지만 저장은 저장대로 삭제는 삭제대로 모여있다는 면에서 심적으로 편안했기 때문에 다른 분들 불편하신게 아니면 이대로 가도 좋을 것 같습니다.

모임 기록 컨트롤러를 개인페이지와 모임페이지로 나누는 것에 대해선 어떻게 생각하시나요?

luke0408 commented 1 month ago

컨트롤러 분리에 대해서는 찬성이지만 오늘 회의 시에 기획 수정관련 이야기가 많이 나왔으니 이부분은 한번 정리하고 적용 방향성을 확정해보죠!