UI/UX
- 회원가입 페이지에서 아래 스크롤을 내려야 회원가입 하는 폼이 나오고, 바로 좋아하는 책의 문구를 입력하라고 나옵니다.
- 회원가입 페이지에서는 바로 화면에 회원가입하는 폼이 나오는것이 좋을 것 같고 굳이 스크롤을 이용하여 내릴 필요가 없을 것 같습니다.
- 좋아하는 책의 문구를 입력하는것은 비회원도 가능한거라면 문구 입력 후 엔터 클릭시 문구 입력이나 버튼 을 넣어줘야 할 것 같아요.
- 로그인이랑 회원가입 화면을 같은 디자인을 사용해도 좋을것 같아요
- 로그인, 회원가입 화면에서 메인으로 돌아가는 방법이 있으면 좋을 것 같습니다.
- 메인에서 새로운 글 작성할때 장르와 업로드 가 아직 선택이 안되는데 기능 구현 전이겠죠..?
- 아직 모든 api 연동 완료 전이겠죠…?
- 아직 모든 api 테스트가 안되는 듯 하네요!
- 상단에 종모양(알림)은 알림메시지 기준이 무엇인가요??
- 예를들면 로그인 한 사용자한테만 필요한 특정 메시지만 알려준다면 종모양은 로그인 하고나서 보여주는게 좋을것 같아요!
- 사용자 검색할때 엔터키나 검색 버튼을 넣어서 입력 완료 후에 api 호출해주는게 좋을 것 같습니다
- 지금은 키보드 입력할때마다 api 호출하고,
- 그리고 404 에러났을경우는 검색된 사용자가 없습니다 정도 보여주는게 좋을 것 같아요
Readme.md 파일에 최종제출 하기 전에 한번 작성해주는게 좋을 것 같습니다.
Line 2:10: 'useEffect' is defined but never used no-unused-vars Line 2:21: 'useState' is defined but never used no-unused-vars 이런 것들은 잡아주는게 좋습니다!
프로젝트 구조
- components / domain 안에 Banner, BookCardView, Form 등의 컴포넌트들이 있는데 이것은 도메인의 성격은 아닌 것 같습니다.
- 도메인의 성격은 Book 정도가 될것 같습니다.
- domain/BookCardView/BookCard, BookListSlider 이런 구조로 되어있는데
domain/Book/Card,domain/Book/List이렇게 구분해주는건 어떨까요…? 북카드 북카드 뭔가 중복되는 느낌입니다. - 전체적으로 components 안에 base 폴더와 다른 컴포넌트들을 도메인 안에 넣지 말고 그냥 flat 하게 구조를 가져가는것도 방법이겠네요.
- Form도 그냥 SignInForm, SignupForm 이렇게 사용하는게 더 명확할것 같고, SubmitButton은 base에 있는 Button컴포넌트로 대체해도 되지 않을까 싶습니다.
기획 & 회고
- keep
- 협업할때 중요성을 알게 된것 같아 좋습니다. (컨벤션, 코드규칙, 브랜치 전략 등)
- 일정에 잘 마무리 할 수 있게 오버엔지니어링인지 아닌지 등 스펙 아웃도 중요하죠!
- 노션이든 지라든 이슈관리하는것은 중요하다고 생각합니다. 나중에는 지라나 다른 툴로서 이슈마다 git commit을 연동하면, 다른 누군가가 히스토리를 더 빨리 파악할 수도 있습니다!
- 저는 개인적으로 저희팀 문화가 좋았다고 생각한것이, 급한 문제 처리, 안건 제안 등 원활한 소통이 이루어졌다고 생각합니다!
- problem
- 역할 분리같은 경우는 많은 연습이 필요하리라 생각 듭니다!
- 짧은 일정에는 불필요한 소통 시간을 줄이는것이 중요하다고 생각합니다. 그렇기때문에, 정확한 기획 디자인, 설계가 필요합니다
- 회고록에 남긴것처럼 컨퍼런스처럼 따로 회의록이나 중요한 내용을 문서로 작성해서 관리해두는 것이 좋습니다.
- 그래서 문서작성도 개발 잘하는것 만큼 중요합니다.
- PR 단위가 커서 부담감이 있었따면, PR을 잘꺠 쪼개는 연습을 충분히 해야 할 것 같습니다. 그리고, 이 PR이 어떤 역할인지 템플리셍 설명을 자세히 적어주는게 좋습니다. (단 커밋 내용을 또 장황하게 쓸 필요는 없다고 생각합니다.)
- try
- 확실히 짧은 시간 안에는 정확도 높은 기능, 선택과 집중, 스펙아웃 등 정해야 할 것들이 많습니다. 이번 프로젝트로 인해 많은것을 느끼셨다니 다행입니다.
- 리팩토링은 꼭 추천드립니다!
- 서로 솔직하게 피드백을 주고받고, 컨디션을 체크해주면서 팀원끼리 으쌰 하는것도 중요하죠!
- PR리뷰가 부담이었다면 특정 받고 싶은사람을 선택하거나, 최소 한명만 approve 하고 일단 머지하자 라는 방향으로 진행해도 괜찮앗을것 같습니다.