invisible-hands / frontend

https://betting-ground.netlify.app
2 stars 0 forks source link

[Feat] 프로젝트 고도화! 뭘 해야 하냐면요... #63

Open GHeeJeon opened 10 months ago

GHeeJeon commented 10 months ago

완성도 높이기

배포

코드리뷰

GHeeJeon commented 10 months ago

코드리뷰

Preview 시리즈

메인페이지에 진입하자마자

  1. useEffect 를 이용해 요청 받고 응답을 화면에 보여줌
  2. 인기 상품, 마감 임박, 신규 상품 화살표 버튼 클릭, 해당 페이지 이동
  3. hot, deadline, new 키워드 수정 필요? 백 코드에 쓰이는 변수명과 통일 필요성?
  4. axios 코드 스타일 통일 필요?

AuctionRecord

  1. axiosInstance 설정, 프로젝트 내 axios 를 사용하는 모든 컴포넌트에 적용된다면, index.js 또는 App.js 에서 정의하는 건?
  2. console.log('필터 되냐?'); 와 같은 로그 출력코드 삭제 필요
  3. auctionStatusOptions 에서 '': '전체' 로 정의, AUCTION_TOTAL 로 정의했을 때와의 차이점?
  4. Auction, Selling, Purchase 에서 날짜 처리하는 부분을 utils 로 따로 분리하는 건 어려울지?
  5. 조회 시작 날짜와 끝 날짜가 오늘로 동일, useEffect 로 오늘 기록을 디폴트로 보여주고, 시작/끝 드롭다운 으로 값 변경해서 보여줌
  6. 두 개의 useEffect 를 하나로 합치는 건?
  7. ~&size=8 대신 ~&size=itemsPerPage 로 사용했을 때와의 차이점?
  8. // 다른 페이징 관련 상태도 필요하다면 여기에서 업데이트 ? 자세한 설명 필요

정리 너무 오래걸린다 한꺼번에 모아두자

  1. createAuction 등 몇몇 함수가 사용되지 않음, 사용 가능하게끔 수정 예정? 삭제 예정?
  2. useEffect vs react-quary
  3. 모달 별 handleClickOutside 유무 차이점?