# Group 2 peer session
## 참여자
- J003 강동훈

- J121 우승진
- J193 조준형
- J210 최홍일

## 진행
- 자기소개
- 시연 및 과정 공유
- 코드리뷰
- 회고
## 코드리뷰
### J003 강동훈
- pug 템플릿 엔진을 사용하실 계획인가요?
- 아니요..지워야 합니다!
- db 에 create 하실때 담아야할 변수가 많다면 객체로 한꺼번에 넣어주면 좋을꺼같아요.
- 개인적으로는 이게 보기 편해서 그렇게 했는데 그렇게 하는게 맞는거같습니다
- model , models 폴더가 두개 있어서 혼란이 조금와서 네이밍을 좀 다르게해주시면 좋을꺼같아요.
- model 폴더에 있는걸 models로 옮기거나 다르게 분류하고 싶었는데 에러나서 실패해서 아직 못고쳤습니다 ㅠ
- status 코드를 분리한거 좋네요~
- 😁
- sequelize config 파일은 보안이슈때문에 ignore 하는게 좋을 것 같아요!
- 넵 이번에 .env를 처음 활용해봤는데 그것도 해봐야겠어요!
- router와 service를 따로 분리하신 점이 저랑 다르네요. 한 번 고민해봐야겠어요
- 😁
### J121 우승진
- 디렉토리 구조가 너무 깔끔해서 좋네요.
- 디렉토리 구조는 항상 고민되는데, 아직 프로젝트가 좀 작아서, 줄이려고 노력했습니다!
- 파일 이름을 a.b.js 이렇게 하는게 보기 좋네요. 배웠습니다
- 저도 다른분 코드보고 좋은점 같아서 배웠습니다 :+1:
- 저번 코드리뷰때? 받은 리뷰사항이기도하고, 크롱님이 말씀하셨던걸로 기억하는데, FE, BE 각각 `module package` 관리를 따로해주는 것도 좋은 것 같아요. 아까 말씀하시기론 웹팩도 FE에만 적용하신다고하니 한 번 고려해보시면 좋을 것 같아요
- 적용해보고 싶었는데 시간이랑 파보고 싶은 부분이 많아 아직 적용해보진 못했네요. 다음번에 적용해보도록 해보겠습니다!
- config 디렉터리를 따로 관리하는 방법은 생각도안하고 있었는데, 되게 깔끔하고 좋네요!
- 환경 설정 할일이 이제 많이 생기다 보니 폴더로 관리하면 좋은 것 같아요 :)
### J193 조준형
- 라우터에서 GET이랑 DELETE 이렇게 한번에 메서드 묶어 쓸수 있는거 이제 알았네요 ㄷㄷ :+1:
- 저도 리뷰해주시는 분이 알려주셔서 배웠습니다!
- 응답에서 FAIL이면 응답 코드를 200이 아니라 다르게 보내는게 좋지 않을까요..?
- 깜빡했네요! 아직 예외처리같은 부분은 작성하지 못해서 추후에 함께 반영하도록 하겠습니다!
- 서비스를 로직을 잘게 쪼개셔 사용하신게 보기 좋았어요. :+1:
- ps. 서비스와 컨트롤러 구분이 항상 모호하다고 생각했는데, 덕분에 조금 기준이 생겼네요 참고해서 저도 반영해봐야겠어요
- 감사합니다!
- `readme`에 api를 잘 정리하셨네요. 프로젝트 진행하면서 도움이 많이 될 것 같아요. 개인적으로 `markdown`에 표 적용하는건 정말 어렵던데 배우고싶네요 ㅎㅎ
- 타이포라에서 작성하면 쉽게 만들수 있어요 ㅋㅋ

### J210 최홍일
- `breakdown`은 어떤 역할인가요?
- `breakdown` 은 내역 입니다. 내역관련된 api 처리하는 라우터이름이에요!
- service나 controller 디렉터리를 만들어서 서비스 로직을 처리하는게 좋을거같아요
- 현재 디렉터리 구조상 router - service 로 서비스 로직을 처리하고 있기는한데 아직 디렉터리 부분은 어렵네요
- read me 를 깔끔하게 정리하신 부분이나 프로젝트로 마일스톤 및 이슈를 관리한점이 보기 좋았어요.
- 관리는 잘하려고하지만 항상 관리가 잘 되지는 않네요 감사합니다 😀
- 시퀄라이즈 config는 gitignore하신건가요?
- 네 이그노어 시켰습니다. ncloud db에 직접 접속하다보니 혹시모르는 사태를 예방하기 위함입니다 ☢☣

## 회고

### J003 강동훈
- env를 적극 활용해서 repo에 민감한 내용이 올라가지 않도록 하자
- 나도 readme 이쁘게 해야겟다 ㅠ
- 라우터 체이닝을 활용해보면 좋겠다
- 언제나 프로젝트 관리가 잘 되지가 않는다. 의식하는 것 필요

### J121 우승진
- 로직을 잘게 쪼개서 적용하자.
- read me 관리 잘하면 멋있어보인다.
- 프로젝트 마일스톤 및 이슈관리 꼼꼼히 하면 보기 매우 좋아보인다.
- 마지막까지 열심히하자...

### J193 조준형
- 디렉토리를 잘 관리하는 다양한 방법에 대해 고민할수 있었다.
- 깃헙의 기능을 잘 활용하자
- 커밋 관리를 잘해야겠다.

### J210 최홍일
- 타이포라써서 리드미를 표로 정리하면 좋을 것 같다.
- service - controller 로직에 대해서 많이 배울 수 있었다.
- 라우터에서 메서드체이닝을 사용하면 좀더 깔끔하게 볼 수 있다.
- 지난 미션까지도 배포는 신경을 잘 못썼는데 좀 신경써봐야겠다.
- 멘탈 잘 잡고 끝까지 완주하기