Skip to content

Conversation

@tnals0924
Copy link
Member

@tnals0924 tnals0924 commented Mar 3, 2025

  • 네비게이션 탑바가 있는 레이아웃과 없는 레이아웃으로 분리하였고, 데스크탑 루트 레이아웃에 react-hot-toast 추가

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

  • 새 기능 추가
  • [] 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 리팩토링

Check List

  • 🔀 PR 제목의 형식을 잘 작성했나요?
  • 🏗️ yarn build는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요? (TODO, 주석, clg... etc.)
  • ✅ 컨벤션을 지켰나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 🙇‍♂️ 리뷰어를 지정했나요?

✅ Key Changes

이번 PR에서 작업한 내용을 간략히 설명해주세요

  1. 데스크탑에 네비게이션 탑 바를 추가하였습니다. 이를 위해 (nav-bar) 디렉토리 추가 후 레이아웃을 따로 생성했습니다!
  2. DesktopLayout에서 useAuthRedirect() 훅 사용 시에 /mobile/sign-in 으로 이동하던 현상을 수정했습니다.
  3. react-hot-toast 라이브러리 적용했습니다. 아래 스크린샷에서 확인해주세요.

📢 To Reviewers

2번의 원인이 페이지 로드와 동시에 API 요청을 보내게 되는데 PrivateAxiosInstance의 intercepter에서 토큰 검증 후에 모바일 로그인 화면으로 이동시키는 것이었습니다.
일단 임시 방편으로 intercepter에서도 pathname에 따라 desktop은 desktop으로, mobile은 mobile로 이동하도록 했습니다.
이렇게 수정하면 문제점이 useAuthRedirect()에서도 이동시키고, intercepter에서도 이동시키기 때문에 똑같은 주소로 2~3번 이동시킨다는 문제가 있습니다. 추후에 로직 갈아엎어야 할 거 같네요.

📸 스크린샷

image image

🔗 참고 자료

- 네비게이션 탑바가 있는 레이아웃과 없는 레이아웃으로 분리하였고, 데스크탑 루트 레이아웃에 react-hot-toast 추가
@tnals0924 tnals0924 added 🐛 BUG 버그를 고친 경우 ✨ FEAT 기능 구현 labels Mar 3, 2025
@tnals0924 tnals0924 self-assigned this Mar 3, 2025
@tnals0924 tnals0924 linked an issue Mar 3, 2025 that may be closed by this pull request
4 tasks
@vercel
Copy link

vercel bot commented Mar 3, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 3, 2025 0:35am

@github-actions github-actions bot added the 🚀 API API label Mar 3, 2025
Copy link
Member

@hyeonjin6530 hyeonjin6530 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다!!

@tnals0924 tnals0924 merged commit 74ed924 into develop Mar 3, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 BUG 버그를 고친 경우 🚀 API API ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 데스크탑 레이아웃 추가

3 participants