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.

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

#225 : 로그아웃 API 구현해요 #228

Merged
merged 9 commits into from
Jan 31, 2025
Merged

Conversation

jeongjaino
Copy link
Member

@jeongjaino jeongjaino commented Jan 26, 2025

1. 📄 관련된 이슈 및 소개

#225 : 로그아웃 시에 알림 받지 않도록 수정해요.

2. 🔥 변경된 점

1. 로그아웃 API 가 추가되었습니다.

  • 서버에서 토큰을 계속 지니고 있어서, 로그아웃된 이후에도 알림이 오는 이슈가 있었는데요.
  • 이를 서버에서 처리하기로 결정하였고, 로그아웃시, 해당 API를 호출하도록 수정했습니다 .

2. 회원탈퇴 로직 수정

  • Kakoa SDK UserApiClient 내에 unlink 메소드가 있는 점 확인하였고, 해당 메소드를 저희는 쓰지 않고 있더라고요 ?
  • 해당 메소드는 해당 계정이랑, 카카오 서버랑 연결을 끊기 + logout(토큰 만료)의 기능을 가지고 있고, 탈퇴시에 사용하는 것 같아요.
  • 관련 문서

3. BaseViewModel Channel

  • Channel 내에 Buffered 속성을 추가했습니다 .
  • 테드팍 선생님 블로그 참고했는데요. 해당 속성을 추가하지 않으면, 구독자가 나타날 때까지 다음 이벤트를 호출하지 않아요.
  • 해당 속성을 추가하면, 이벤트 버퍼에 이벤트를 추가하고, 다음 이벤트로 넘어가요 !

4. 탈퇴 / 로그아웃 UI

  • 탈퇴 / 로그아웃 API 수정
  • 모달내에 버튼 클릭시, 모달을 닫고 로딩 애니메이션 보여주도록 수정 + 실패 시 토스트 노출하도록 수정했습니다.

3. ✅ 필수 체크 사항

수정된 회원탈퇴 / 로그아웃 상태에서 정상적으로 동작함을 확인했습니다.

image

로그아웃 상태에서 알림 오지 않는 점 확인했습니다.

  • 버전은 1.3.0으로 업데이트하고, 어드민 앱에서 알림 보내면 수신할 수 있는데요 !
  • 로그아웃 상태에서는 알림이 오지 않고, 로그인 상태에서는 알림이 오고 있는 점 확인했습니다.

4. 📸 작업물 사진 공유(선택)

로그아웃 썡공

logout1.webm

로그아웃 실패

logout_fail.webm

5. 💡알게된 혹은 궁금한 사항

@jeongjaino jeongjaino added 🌱기능🌱 새로운 기능을 추가해요 ! 🍻제이콥🍻 24기 정진호 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요? labels Jan 26, 2025
@jeongjaino jeongjaino self-assigned this Jan 26, 2025
@jeongjaino jeongjaino changed the title #225 : 회원탈퇴 API 구현해요 #225 : 로그아웃 API 구현해요 Jan 26, 2025
@jeongjaino jeongjaino linked an issue Jan 26, 2025 that may be closed by this pull request
1 task
Copy link
Contributor

@flash159483 flash159483 left a comment

Choose a reason for hiding this comment

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

우왕

@kpeel5839
Copy link

우왕

자기야 서버드리븐 API도봐줘

@jeongjaino jeongjaino merged commit d531fd3 into develop Jan 31, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱기능🌱 새로운 기능을 추가해요 ! 🍻제이콥🍻 24기 정진호 🔥리뷰 기다리는 중🔥 PR을 올리고 코드리뷰를 기다리고 있는 상태입니다. 🔧리팩터링🔧 리팩터링일까요 리팩토링일까요?
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WS-460 [FEATURE]: 로그아웃 API 구현해요
3 participants