재호팀 → 오프팀
- 햄버거 버튼의 로그인/로그아웃 제대로 작동 안함 (햄버거 메뉴 내 버튼 기능 잘 안되는 듯함)
- 햄버거 버튼을 누르고 ‘스토리 작성’ 버튼을 누르면 화면 전환이 되지 않음
- 회원가입 성공여부 파악 불가능!
- 스토리 쓰기 에러
- 스토리 들어갔을 때, 헤더에 유저 정보가 가려진다.
- 하나의 파일이 그렇게 길지 않아서 보기 좋네요.
재호팀 → 오프팀 (코드 리뷰)
- 승환
:
validateSignInInput 좀더 분리시키면 보기 편할 것 같습니다. 나머지는 넘- 은지
- 지연
: 엄청 깔끔한 폴더구조.. 멋있다고 생각했고, hook으로 잘 빼서 정리된 부분이.. 매우 인상 깊었습니다. 잘 배우고 갑니다.. ㅎㅎ
- 성현: 구조적으로 너무 좋네요. 많이 배우고 갑니당
- 동우
- 너무 배울점이 많은 코드네요
- 이렇게 나누는 기준이 어떤건지 궁금합니다!
오프팀 → 재호팀
- 상태 관리가 10여개?!
- 타입스크립트도 깔려있고 readme에 적혀있는데 타입스크립트 파일이 없어요.
- 추후에 반영되는건가요?
- input password type (비밀번호 노출)
- 로그인창 필드 name 로그인 → email로 변경되면 의미가 더 명확해지지 않을지
- Router에서 exact 빼도 되지 않을까유
- style.js로 style 코드는 빼니까 컴포넌트 코드는 깔끔해서 보기 좋았습니다.
- 컴포넌트 세분화와 상수화가 잘 되어 있어요!
- 이미지 드래그앤드롭 기능까지 구현되어 있어서 좋아용
- 컴포넌트별로 사용하지 않는 state가 종종 있는 것 같아요
- 게시글 추가하고 나서 게시글을 확인하는 모달이나 페이지를 보여주면 좀 더 자연스러울 것 같습니당
- UI 너무 예뻐요!!!! 믓찌다
- 검색 자동완성창을 하나하나 다 만드셨군요:)b
- TypeError 발견!
- 사용하지 않는 변수
- import가 너무 많은것 같아요 (./style) 처럼 하나의 파일에서 import하는식으로 어느정도 정리가 가능할것 같아요
- 제 개인 취향인데 다음 경우 그냥 lazy만 적어도 true가 들어가서 더 간결하게 작성할 수 있을 것 같아요!
- axios instance 생성할 때 interceptor를 사용하니까 코드 이해하기 더 쉽네요. 하나 배워가요!
- fragment 빼도 될 듯욥!
- 함수를 합쳐볼 수도 있지 않을까요?
if (keyword.length > 0) { searchAll(encodeKeyword(keyword)) .then(({ data }) => setSearchResult(filterdPost(data))) .catch(console.error); ... } 약간 이런식?