코드 리뷰시 보는 항목
- 성능
- 가독성
- 최적화
튜터님 피드백
공통적 적용할 부분
- favicon 지우기 O
- PR 템플릿 체크리스트 적용 && .github 폴더로 옮겨주기 O
- PR 시에 체크리스트 제대로 작성안한 지적 O
- supabase gen script 에 고유값 env 파일로 이동 X
- 에러처리 고민 에러 페이지, 로딩 페이지 디자인 요청하기 O
- 공통 컴포넌트로 최대한 사용 후 나중에 공용 컴포넌트만 변경하기
- default 브랜치 dev으로 하지 않는게 좋을 것 같습니다
default 브랜치를 변경하고 PR merge 했을 시에 어디로 합쳐지는지? [최종 전에]
- html lang=ko로 적용하기 O
개별 피드백
- "data"라는 변수명은 변경해주세요 → data, error 이렇게 받아올때 맞는 변수명으로 변경 요망 (양성훈 완료)
- import 절대경로로 모두 수정해줍시다 (상위 폴더가 같은 시에는 상대경로로 해도 됨)
- 에러 처리: alert처리 혹은 모달을 띄워주는 걸로
- admin.ts 라는 파일의 이름이 명확하지 않은 것 같습니다 → 회원 탈퇴 기능에 필요합니다! (안 지울 예정)
- Header에 client에 해주는게 좋을 것 같습니다 → 박감자가 할 예정