유저 닉네임이 4글자 이상 넘어갈 때 헤더에서 줄바꿈되는 버그가 있습니다

이슈개요

• 사용자 상태 랜더링(유저 네임)이 로그인시 변경되어야 하는데, 로그인 후에도 변경되지 않는 현상

상세내용

• 로컬스토리지에 저장된 usename을 이용해 사용자 상태의 유저네임을 변경해 줍니다.
• 아래 코드와 같이 로컬스로리지에 데이터를 불러오고 난 뒤 setIsLoggedIn(true)로 변경해주어 사용자 상태 렌더링을 변경해주는 로직입니다.

useEffect(() => {
    const storedResponse = localStorage.getItem('response');
    if (storedResponse) {
      const parsedResponse = JSON.parse(storedResponse);
      const username = parsedResponse.user.username;
      setUser({ userName: username });
      setIsLoggedIn(true);
    }
    console.log(storedResponse);
  }, []);

{isLoggedIn ? (
          <>
            <Link to={`/mypage`}>{user?.userName}님</Link>
            <Link to="#" onClick={handleLogout}>
              로그아웃
            </Link>
          </>
        ) : (
          <>
            <Link to={`/login`}>로그인</Link>
            <Link to={`/signup`}>회원가입</Link>
          </>
        )}

그러나 로그인 뒤에도 상태 변경이 일어나지 않습니다.

새로고침을 해야만

로 변경되는 이슈가 발생했습니다.

Overview

1. 심부름 서비스를 센스있게 잘 구현하셨습니다.
2. 리드미 API Table에 API_URL은 꼭 기입되면 좋겠습니다.
3. 프로젝트 문서에서 사용기술은 가능하면 기술스택 고유의 아이콘 포함해서 작성해 주시면 더 좋습니다.
4. 좋아요에 optimistic updates 적용하셨는데, onError과 onSettled가 없네요. 에러가 발생했을때의 예외처리까지 신경써주시면 더 좋습니다. 뿐만 아니라 좋아요를 연속으로 클릭하는 사용자를 대비해서 디바운싱까지 적용해보면 더 좋겠습니다.
5. PR 리뷰도 리뷰어 지정해서 활발하게 잘 진행하셨습니다.
6. 회원가입 시 회원가입 버튼 클릭 전에 “이미 존재하는 이메일”인지 알아볼 수 있도록 API요청하는 로직을 넣으셨는데, 이메일 형식 조건을 만족시키면 타이핑할때마다 요청이 가기 때문에 별도로 중복 확인 버튼을 따로 만들거나, 이대로 진행하고 싶으시면 디바운싱을 적용하시면 더 좋겠습니다.
7. 부탁 내용 작성 시 지도 클릭보다는 좀더 정확한 주소입력을 위해서 실제 주소 API를 통해서 주소입력을 하면 더 좋을 것 같습니다.
8. 무한스크롤 기능 useInfiniteQuery로 잘 구현해주셨습니다.
9. 신고하기 기능이 있는데, 관리자 페이지도 따로 있으면 좋겠습니다.
10. json-server-auth 활용하셔서 jwt을 이용한 인증인가 처리 잘해주셨습니다.
11. 타입스크립트 사용이유: 컴파일하기전에 타입에러를 잡아주는 게 아니라 런타임전에 타입에러를 잡아줍니다.
12. 로그인버튼 비활성화 로직 좋습니다.
13. 프로젝트 관리 정말 중요합니다. 팀장님이 매 회의마다 각 기능의 마감기한 및 전체 진도율을 체크해주셔야 합니다.

이슈 개요

• NotFoundPage 에서 footer 를 숨기려고 했으나 숨겨지지 않음

상세 내용

NotFoundPage 경로에서 footer를 숨기려고 시도.

그러나 와일드카드 문자로 이루어진 경로 에서는 적용되지 않는 문제