-
Notifications
You must be signed in to change notification settings - Fork 1
2주차 팀 회고
박상혁 edited this page Nov 18, 2022
·
1 revision
- 로컬로그인과 4개의 SSO를 포함한 모든 로그인 API를 구현했다.
- 인프라 제외 스프린트 때 세웠던 서버의 개발 계획은 지켰다고 볼 수 있다.
- 우리만의 서버 DDD 구조를 잘 확립해 놓았다.
- Request Body는 Request DTO와 Body decorator를 이용해 자동 매핑
- request dto 내에서 class-validator로 간단한 validation 진행
- Service Layer에서 domain 클래스 적극 활용 (DDD의 핵심, 아직 미숙하다)
- Repository method는 domain class를 인자로 받고 domain class를 return 하는 것을 기본으로 함 (이전 프로젝트 경험 상 이를 지키기 애매한 경우가 몇 번 있었는데 이럴 때는 유도리에 있게 넘어갑시다.)
- 유저에게 주는 input은 1차적으로 response dto 사용
- 변환 및 간단한 검증 로직은 response dto 내에 존재
- dto가 단순 데이터 덩어리가 아니게 되지만 개인적으로 생성자 내에 변환, 검증 로직이 있는 것은 괜찮다고 봄
- 이로써 파라미터가 간결해지며 응답 사항에 수정할 필요가 생길 시 대부분의 경우 dto 파일만 수정해서 처리 가능하다는 장점이 존재한다.
- 변환 및 간단한 검증 로직은 response dto 내에 존재
- Request Body는 Request DTO와 Body decorator를 이용해 자동 매핑
- 저희조 개발 속도 빠른편인것 같습니다
- 그치만 1등 달려보고싶습니다
- Human Problem (예비군, 코로나) 이슈로 서버 개발이 늦어졌다.
- 본래 계획했던 인프라 및 CI / CD 파이프라인 구축이 늦어졌다. TT
- 유저 테이블을 나눈 것이 과연 잘 나눈 것인가?
- TS에서 상속 기능이 생각보다 사용하기 불편하다.
- method overloading이 불가능하기 때문
- 유저 조회 시 join이 발생하는 경우가 매우 많다.
- 어떻게 할 것인가… 아직 제대로 정하지 못했다. 추후 계속 진행하며 정할 계획이다.
- TS에서 상속 기능이 생각보다 사용하기 불편하다.
- 기획이 생각보다 많이 변경되었다.
-
pr approve 속도가 늦어 개발진행이 늦어지는감?이 살짝 있습니다
⇒ review 늦게달리면 그냥 강제머지
⇒ 스탭있는 일은 due to 를 늦게 잡는것이 좋을듯
⇒ 슬랙에 pr요청 올리자
⇒ 아니면 강제머지하자
⇒
-
CI / CD 빠르게 끝내자~ 이말이야
-
서버 DDD 구조를 어느정도 확립해 놓았기 때문에 이제 빠른 기능 개발이 가능하겠다.
-
Test 코드도 신경써야한다 이제!
-
좀 더 늦게까지 개발하자~
- 서재완에게 해당하는 말