manito42 / backend

42 Manito Backend Repository
https://42manito.com
3 stars 2 forks source link

116 feature 멘토링 취소 사유 추가 #119

Closed koreanddinghwan closed 1 year ago

koreanddinghwan commented 1 year ago

resolves #116

koreanddinghwan commented 1 year ago

잘 봤습니다. Repository 들을 PrismaModule에서 통합으로 가지고 있게 변경한 것 맞죠?

cancel 쪽은 별 문제 없어보입니다!

네 맞슴다 model별로 가지고 있을까 아니면 prisma module에서 모두 가지고있을까 고민했는데, prisma module의 역할을 repository 초기화 용으로 사용하는게 좋을 것 같아서 이렇게 구조화했습니다.

koreanddinghwan commented 1 year ago

잘 봤습니다. Repository 들을 PrismaModule에서 통합으로 가지고 있게 변경한 것 맞죠? cancel 쪽은 별 문제 없어보입니다!

네 맞슴다 model별로 가지고 있을까 아니면 prisma module에서 모두 가지고있을까 고민했는데, prisma module의 역할을 repository 초기화 용으로 사용하는게 좋을 것 같아서 이렇게 구조화했습니다.

그리고 model별로 가지고있으면, repository가 필요한 module들에서 model을 import해야하는 상황이 와서 다소 복잡해지는 경향이 있었습니다. 그래서 prisma module에 repository 생성책임을 모두 넘겼습니다