중간점검 총평
- 최대한 기능에 집중해주세요. 디자인 및 세부적인 오류들을 잡는데 너무 시간을 많이 쓰지 마세요.
- 지금 PR에 기술한 내용들이 구체적이어서 좋습니다. 이미지까지 첨부해서 작업한 부분들에 대해 시각적으로 보여주는 건 어떨까요?
- storybook에 너무 많은 시간을 사용하지 마시고 진행해주세요. 지금은 의미가 없어보이고 진행상황을 봤을때 시간도 촉박합니다
- testing도 지금은 넘어가주세요. 굳이 필요하다면 login만 해도 무방합니다.
- hook을 사용해서 로직 분리를 잘 하신 점은 좋아보입니다.
Git commit 제목은 명령조
로
커밋메시지는 보통 명령형으로 사용합니다.
api 불러오는 로직의 추상화와, 에러처리가 아쉽습니다.
- axios 글로벌단위에서 interceptor를 활용하면 어떨까요
Readme에 실행방법 추가하기
제대로된 프로젝트들은 실행방법이 나와있습니다.
ex) Gatsby 라이브러리
You will need a new or existing Contentful space to use this starter and will be asked for your Space ID, Content Management API Key (also referred to as a Personal Access Token) and Content Delivery API Key during installation.
- Create a Gatsby site
Use the Gatsby CLI to get started locally:
npx gatsby new my-homepage https://github.com/gatsbyjs/gatsby-starter-contentful-homepage
- Run the Contentful setup script
From your site's root directory, run:
cd my-homepage
yarn setup
This will run a script to populate your Contentful space's content model and add demo content.
- Start developing
In your site directory, start the development server:
yarn start
Your site should now be running at http://localhost:8000
- Open the source code and start editing
회원가입 Form을 UX에 맞게 변경해보기
label은 보통 위에 있는 것이 일반적입니다.
PR시 코드리뷰가 너무 없음
- 지금 너무 바쁜 것인지, PR에 코멘트가 너무 없습니다.