Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FEAT] 로그아웃 api 연결 #181

Merged
merged 3 commits into from
Jul 18, 2024
Merged

[FEAT] 로그아웃 api 연결 #181

merged 3 commits into from
Jul 18, 2024

Conversation

seong-hui
Copy link
Member

작업 내용 🧑‍💻

  • 로그아웃 시 localstorage를 지우고 로그인 페이지로 redirect 되도록 하였습니다.
  • accessToken과 refreshToken이 둘 다 만료 또는 유효하지 않을 시 "로그인이 만료되었습니다. 다시 시도해주세요." 라는 alet이 뜨고 로그인 페이지로 redirect 되도록 코드를 수정하였습니다.

알게된 점 🚀

기록하며 개발하기!

리뷰 요구사항 💬

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • 폴더구조 변경 혹은 더 추가로 로그인페이지로 리다이렉트가 필요한 부분이 있다면 말씀해주세요!

관련 이슈

close #167

스크린샷 (선택)

Screen.Recording.2024-07-18.at.12.02.46.AM.mov

@seong-hui seong-hui requested review from wrryu09, Kjiw0n and jeeminyi July 17, 2024 15:03
@seong-hui seong-hui self-assigned this Jul 17, 2024
Kjiw0n
Kjiw0n approved these changes Jul 18, 2024
View reviewed changes
Copy link
Contributor

@Kjiw0n Kjiw0n left a comment

Choose a reason for hiding this comment

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

LGTM 👍 👍

@Kjiw0n Kjiw0n merged commit 1217543 into develop Jul 18, 2024
2 checks passed
@Kjiw0n Kjiw0n deleted the feat/#167/logout-api branch July 18, 2024 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kjiw0n Kjiw0n Kjiw0n approved these changes

@wrryu09 wrryu09 wrryu09 approved these changes

@jeeminyi jeeminyi Awaiting requested review from jeeminyi

Assignees

@seong-hui seong-hui

Labels
🏰 성희 🔗 api 연결
Projects
@DAWM-FE
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 로그아웃 api 연결
3 participants
@seong-hui @wrryu09 @Kjiw0n