Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
rjc1704 opened this issue Aug 16, 2023 · 0 comments
Open

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

rjc1704 opened this issue Aug 16, 2023 · 0 comments

Comments

@rjc1704
Copy link

rjc1704 commented Aug 16, 2023

Overview

  1. 심부름 서비스를 센스있게 잘 구현하셨습니다.
  2. 리드미 API Table에 API_URL은 꼭 기입되면 좋겠습니다.
  3. 프로젝트 문서에서 사용기술은 가능하면 기술스택 고유의 아이콘 포함해서 작성해 주시면 더 좋습니다.
  4. 좋아요에 optimistic updates 적용하셨는데, onError과 onSettled가 없네요. 에러가 발생했을때의 예외처리까지 신경써주시면 더 좋습니다. 뿐만 아니라 좋아요를 연속으로 클릭하는 사용자를 대비해서 디바운싱까지 적용해보면 더 좋겠습니다.
  5. PR 리뷰도 리뷰어 지정해서 활발하게 잘 진행하셨습니다.
  6. 회원가입 시 회원가입 버튼 클릭 전에 “이미 존재하는 이메일”인지 알아볼 수 있도록 API요청하는 로직을 넣으셨는데, 이메일 형식 조건을 만족시키면 타이핑할때마다 요청이 가기 때문에 별도로 중복 확인 버튼을 따로 만들거나, 이대로 진행하고 싶으시면 디바운싱을 적용하시면 더 좋겠습니다.
  7. 부탁 내용 작성 시 지도 클릭보다는 좀더 정확한 주소입력을 위해서 실제 주소 API를 통해서 주소입력을 하면 더 좋을 것 같습니다.
  8. 무한스크롤 기능 useInfiniteQuery로 잘 구현해주셨습니다.
  9. 신고하기 기능이 있는데, 관리자 페이지도 따로 있으면 좋겠습니다.
  10. json-server-auth 활용하셔서 jwt을 이용한 인증인가 처리 잘해주셨습니다.
  11. 타입스크립트 사용이유: 컴파일하기전에 타입에러를 잡아주는 게 아니라 런타임전에 타입에러를 잡아줍니다.
  12. 로그인버튼 비활성화 로직 좋습니다.
  13. 프로젝트 관리 정말 중요합니다. 팀장님이 매 회의마다 각 기능의 마감기한 및 전체 진도율을 체크해주셔야 합니다.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant