feat : NavBar 마이그레이션

[Mayreeel]

🛠️ 변경사항

최상위 layout.tsx에서 children을 wrapping하는 방식으로 구현
pathname으로 예외처리 및 useEffect 의존성 배열 변수 할당

☝️ 유의사항

👀 참고자료

❗체크리스트

• 하나의 메소드는 최소의 기능만 하도록 설정했나요?
• 수정 가능하도록 유연하게 작성했나요?
• 필요 없는 import문이나 setter 등을 삭제했나요?
• 기존의 코드에 영향이 없는 것을 확인하였나요?

[github-actions]

Unit Test Results

86 tests  ±0   86 ✔️ ±0   4s ⏱️ -1s
17 suites ±0     0 💤 ±0 
17 files   ±0     0 ❌ ±0 

Results for commit 8e4fc8a. ± Comparison against base commit 5551cf1.

[Yujin-Baek]

NavBar 컴포넌트를 최상위 layout 파일에 추가하셨네요
NavBar는 대부분의 페이지에서 사용되니까 그렇게 하는 게 좋을 것 같네요
NavBar를 사용하지 않는 페이지에서는 props를 이용해서 사용하지 않도록 하면 될 거 같아요 좋습니당 😄
코드 모두 확인했습니다! ✅
수고하셨습니다 🙌

[Mayreeel]

NavBar 컴포넌트를 최상위 layout 파일에 추가하셨네요 NavBar는 대부분의 페이지에서 사용되니까 그렇게 하는 게 좋을 것 같네요 NavBar를 사용하지 않는 페이지에서는 props를 이용해서 사용하지 않도록 하면 될 거 같아요 좋습니당 😄 코드 모두 확인했습니다! ✅ 수고하셨습니다 🙌

Props로 Setter 함수를 내리지 않는 한 NavBar 컴포넌트 안의 값을 변경할 수 없어서 대신 usePathname으로 현재 주소 값이 NavBar를 사용하지 않는 주소면 NavBar가 보이지 않게 하도록 설계했습니다!