Closed SieunKiim closed 3 months ago
MR 리뷰 드렸습니다! 확인 구다사이. 잇빠이 고생하셨습니다. 항시 감사드립니다.
Project 등록 및 조회 관련해서 새롭게 정의된 내용이 있어 일부 변경이 필요해보입니다.
혹시 더 필요한 내용이 있을까요?
Project 등록 및 조회 관련해서 새롭게 정의된 내용이 있어 일부 변경이 필요해보입니다.
- [ ] 프로젝트 응답 시 Healthcheck Url 검증 (살았니 죽었지)
- [ ] Participant 정보 호버 값 추가 필요
- [ ] 단건 조회 시 조회수 관리 로직 구성
혹시 더 필요한 내용이 있을까요?
이상무인것 같습니다~!
도메인 모델 안에 엔티티 생성자 빼자니, 걍 코드 대 참사가 일어나버리는데요... 너무 스레기 같아요 일단 지원하는 기능 내용들 먼저 반영하고 이거 같이 코드 봐보죠
어떤 부분이 스레기 같은지 말씀주시면 일단 같이 생각해보겠습니다!
캐싱 적용한 부분이 2개 인데요
추가로, entitiy로 domain model 만들어 주는 생성자 부분은 PR 분리해서 작업하면 어떨지 문의드립니다. 해당 PR 반영이 우선으로 생각되는 부분도 있구요, 스레드가 넘 길어져서 한번 분리도 필요해보여서 말씀드립니다~
프로젝트 도메인 비즈니스 로직 설계