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

자동 로그인 기능 구현 #71

Merged
merged 5 commits into from
Sep 10, 2023
Merged

자동 로그인 기능 구현 #71

merged 5 commits into from
Sep 10, 2023

Conversation

soopeach
Copy link
Member

개요

  • 자동 로그인 기능 구현

변경 사항

  • TokenManager -> TokenRepository
  • 의존성 cycle 문제로 AuthInterceptor 생성자 일부 수정
  • Splash에서 로그인 여부 확인 후 분기처리

@soopeach soopeach added the Merge ASAP 확인 후 바로 머지 해주셔도 됩니다!! label Sep 10, 2023
@soopeach soopeach requested a review from dogdduddy September 10, 2023 07:39
@soopeach soopeach self-assigned this Sep 10, 2023
}, 2000)
lifecycleScope.launch(Dispatchers.Main) {

val validateTokenJob = validateToken()
Copy link
Member Author

Choose a reason for hiding this comment

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

요 변수명 Job이 코루틴 Job같아서 헷갈리는데 혹시 isLoggedIn 과 같은 변수명은 어떨까요?

@soopeach soopeach merged commit 9c12143 into develop Sep 10, 2023
1 check failed
dogdduddy pushed a commit that referenced this pull request Sep 9, 2024
자동 로그인 기능 구현
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merge ASAP 확인 후 바로 머지 해주셔도 됩니다!!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants