TIL - 20230224

- 프로젝트 3주차 금요일 중간 발표 튜터 피드백

1. 메인페이지 - 은행 바로가기 리스트

- 이 부분 삭제되거나 금융 팁, 관련 뉴스 등 사이트의 얼굴인 메인 페이지 상단에 어울릴만한 아이템으로 대체 후 하단으로 빼는 것이 좋아보임. -> 이미 발표 전 점검 때 피드백 받았던 내용. 수정 필요.

2. 서비스페이지 - 상품 비교 모달

- ui 수정. -> 인지하였으나 발표 전 시간 부족.

- 북마크 기능이 있어 차후 마이페이지 등에서 확인 가능하나 카카오톡 공유 등의 기능도 있으면 좋을 것. -> 카카오톡 공유 기능을 추가한다면 어떤 내용을 보여줘야할 지 생각해볼 것.

3. 서비스페이지 - 상품 검색 결과

- 은행 로고 크기 통일성 -> 수정 필요.

4. 서비스페이지 - 예/적금 예측 화면 UI

- 정확히 이해하지 못해 팀원들과 공유 필요.

5. 서비스페이지&마이페이지 - 북마크 목록

- 예/적금 필터링 걸어서 출력. - 작업중이나 시간 부족으로 피드백 전 수정하지 못한 내용. 작업중.

피드백 후 수정 내역 및 버그 해결 등

1. 라우터 정리

- 랜딩페이지 추가로 라우터 페이지 및 헤더 등 각종 페이지 내비게이트 수정.

2. 메인페이지 - 새로고침 시 user 의 displayName 불러오지 못하는 부분 해결하기.

  // 해결 방법 1
  // state 만들어 useEffect 안에 setstate 로 user 값 변경해주니 새로고침해도 불러와짐.
  // useEffect 가 return 후 실행되므로 깜빡이는 현상 발생. 개선 필요함.
  // const [user, setUser] = useState({});
  // useEffect(() => {
  //   onAuthStateChanged(authService, (user) => setUser(user));
  // }, []);
  
  // 해결 방법 2
  // 기존 const user = authService.currentUser 가 새로고침시 displayName 을 불러오지 못하니
  // 세션스토리지에서 key 를 가져와 JSON.parse 안에 넣어줘서 해결. 유즈이펙트보다 쉽고 유즈이펙트 깜빡임도 사라짐.
  // const userSession = sessionStorage.getItem(isLoggedIn);
  // const user = JSON.parse(userSession ?? "");

* 추가 - 에러 해결한 줄 알았으나 비로그인 시 JSON 에러 재발생. 조치 필요. 유즈이펙트 구문으로 임시 조치함.

3. 마이페이지 - 로그아웃 시 에러 발생.