Skip to content

Conversation

@hyeonjin6530
Copy link
Member

📌 관련 이슈번호

🎟️ PR 유형

어떤 변경 사항이 있나요?

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

Check List

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

✅ Key Changes

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

  1. 작업 내용
  • 로컬스토리지에 저장된 정보를 초기화 시켜서 로그아웃 기능을 구현하였습니다.
  • 로그아웃 버튼 클릭시, 로컬 스토리지가 초기화 되며 /mobile/sign-in 페이지로 이동하게 됩니다.

📢 To Reviewers

  • 추후에 zustand를 사용한 방식으로 수정 가능할 것 같습니다!

📸 스크린샷

🔗 참고 자료

@hyeonjin6530 hyeonjin6530 added the ✨ FEAT 기능 구현 label Feb 26, 2025
@hyeonjin6530 hyeonjin6530 self-assigned this Feb 26, 2025
@vercel
Copy link

vercel bot commented Feb 26, 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 Feb 26, 2025 3:37pm

}: SidebarProps) {
const handleLogout = () => {
localStorage.clear(); // 로컬 스토리지 전체 삭제
window.location.href = '/mobile/sign-in'; // 로그인 페이지로 이동
Copy link
Member

Choose a reason for hiding this comment

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

제가 잘 몰라서 그런 거일 수도 있는데 여기서 Next useRouter()의 replace() 함수가 사용이 불가능해서 window.location.href를 이용하신 걸까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

아니요!!!!!!! 와다다 하다보니 냅다 window를 써버렷습니다 수정할게요!

Copy link
Member

@tnals0924 tnals0924 left a comment

Choose a reason for hiding this comment

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

굿

@hyeonjin6530 hyeonjin6530 merged commit 9106171 into develop Feb 26, 2025
2 checks passed
@sinji2102 sinji2102 deleted the feature/#60-logout branch March 5, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ Feat ] 로그아웃 기능 구현

3 participants