[FE] 레벨 4 리팩터링 (전체)

[juunzzi]

배경

스프린트 4에 빠르게 개발하며 생긴 부채들 및 미처 신경 쓰지 못한 부분들을 개선한다. 가능하면 리뷰가 밀린 풀리퀘 항목들에 대하여 리뷰를 재차 진행한다.

진행사항

• [x] #460
• [x] #332
• [x] #328
  • history의 state를 넘겨주는 구조
  • URL 쳐서 들어갔을 때 대응이 안된다는 점
• [ ] CouponDetailPage의 NotFound early return
• [ ] #327
• [x] types/remote 도메인별로 타입 구분하기
• [x] types/client 안 쓰는 타입 정리
• [x] 뒤로 가기 처리
  • replace 처리
• [x] #321
  • 401?
  • 404?
  • onError 로직을 위임하기?
  • 모든 에러를 ErrorBoundary 로 처리하기?
• [ ] Icon 컴포넌트 css Props 받아서 스타일 처리하기
• [ ] Position 제거?
  • 사전 도입 이유 : 구조 파악이 쉽기 때문에 도입했다.
  • 단점
    • 엘리먼트 하나가 더 추가된다.
• [x] Mutation에 Loading 이 들어가는 것 / displayMessage 의 위치
  • UI 로직일까? 비즈니스 로직일까?
  • 뮤테이션 로직 / 비즈니스 로직 / 컴포넌트 로직 어디에 위치해야할까?
• [x] 쿠폰 디테일 페이지에서 필터링 / 스크롤 위치 기억하게 하기
• [x] 회원 랜딩 페이지 에서 비회원 랜딩 페이지 번쩍이는 현상
• [ ] UserSearchModal 애니메이션 리팩터링
• [ ] ScrollToTop 컴포넌트가 아닌 훅으로 분리
• [x] #337
• [x] #338
• [x] #342
• [x] #464