프로젝트 피드백 - 작성자: 류제천 튜터

[rjc1704]

Overview

1. 프로젝트 컨셉과 서비스명 작명 센스가 정말 좋습니다.
2. 컨셉소개 및 시연 먼저하고 다른 부분 말씀해 주신 발표순서 너무 좋습니다.
3. 비밀번호 찾기 기능 너무 좋습니다.
4. 간단 입력창과 디테일 입력창까지 옵션으로 주는 부분 UX적으로 아주 좋습니다.
5. PR 리뷰 꼼꼼하게 수행한 부분 정말 인상적입니다. 굿!
6. 팀프로젝트 할 때 슬랙과 깃헙PR 연동하는 부분 아주 좋습니다.
7. 깃헙 프로젝트 기능으로 태스크관리 아주 잘 활용하셨습니다. 굿
8. react-router-dom의 Outlet 컴포넌트는 router설정이 nested component(자식 컴포넌트)를 의미하여 router 설정 시 Layout 용도로 자주 사용되니 아래 공식문서를 한번 참고해 보시면 좋겠습니다. https://reactrouter.com/en/main/components/outlet

• 코드 리뷰 코멘트는 아래에 남기겠습니다.

[rjc1704]

path 명을 PascalCase 또는 camelCase 둘중 하나로 통일해 주시는 것이 더 좋겠습니다.

[rjc1704]

44번줄 이벤트핸들러를 할당할 때 함수에 넘겨줄 인자가 없는 경우에는 그냥 signInWithGoogle 로 짧게 남겨주시는 게 좋겠습니다.

[rjc1704]

함수의 return 값에 대한 타입 정의가 없는 부분이 조금 아쉽습니다.

위 스크린샷처럼 함수명에 마우스오버했을 때 나오는 것을 call signature 라고 부릅니다.
만약 함수의 return 값에 대한 type을 SelectedExpenseData 라고 정의한 것이 있다면, 아래와 같이 call signature를 이용한 타입정의가 가능하고 함수 타입을 한번에 끝내고 함수 정의는 자바스크립트 코드로 그대로 작성하시면 좀더 깔끔하게 작성하실 수 있습니다.