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

🔀 :: (#477) Logout UseCase 추가 #486

Merged
merged 2 commits into from
Apr 14, 2024
Merged

🔀 :: (#477) Logout UseCase 추가 #486

merged 2 commits into from
Apr 14, 2024

Conversation

baekteun
Copy link
Member

💡 배경 및 개요

  • 로그아웃을 하는 로직을 처리하기 위한 UseCase를 추가해요

Resolves: #477

📃 작업내용

  • Logout용 UseCase을 추가했어요
  • Repository, DataSource에 Logout을 하는 어플리케이션 로직을 추가했어요

🙋‍♂️ 리뷰노트

  • UseCase라는 개념의 약간 미약해질 수 있지만.. Logout로직을 처리하는 UseCase를 따로 하나 만들었어요
  • LogoutUseCase가 동기적으로 처리됨에도 Completable인 이유는 이후 로그아웃 행동에 비동기적 처리가 추가될 수 있어서예요 (e.g. Logout API 추가)

✅ PR 체크리스트

템플릿 체크리스트 말고도 추가적으로 필요한 체크리스트는 추가해주세요!

  • 이 작업으로 인해 변경이 필요한 문서가 변경되었나요? (e.g. XCConfig, 노션, README)
  • 이 작업을 하고나서 공유해야할 팀원들에게 공유되었나요? (e.g. "API 개발 완료됐어요", "XCConfig 값 추가되었어요")
  • 작업한 코드가 정상적으로 동작하나요?
  • Merge 대상 브랜치가 올바른가요?
  • PR과 관련 없는 작업이 있지는 않나요?

🎸 기타

@baekteun baekteun changed the base branch from develop to 477-support-refresh-token April 13, 2024 13:36
Copy link

github-actions bot commented Apr 13, 2024

✅ Successful finished SwiftLint

@github-actions github-actions bot added the ♻️ Refactor 코드 리팩토링 label Apr 13, 2024
Copy link

✅ Assign 자동 지정을 성공했어요!

@baekteun

Base automatically changed from 477-support-refresh-token to develop April 14, 2024 05:03
@baekteun baekteun force-pushed the 477-logout-usecase branch from f8659b3 to 564fc47 Compare April 14, 2024 05:05
@baekteun baekteun merged commit a5edda4 into develop Apr 14, 2024
3 checks passed
@baekteun baekteun deleted the 477-logout-usecase branch April 14, 2024 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature레이어에서의 Keychain을 직접 사용 코드 리팩토링
3 participants