-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat [#104] 코디네이터 구현 #110
base: develop
Are you sure you want to change the base?
Feat [#104] 코디네이터 구현 #110
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
코디네이터.. 깔끔하고 클린.. 수고하셨습니다 👍🙇
private func showSplashScreen() { | ||
currentView = AnyView(SplashView(coordinator: self)) // 코디네이터 주입 | ||
|
||
/// 스플래쉬에서 토큰 검사 과정 (혹은 홈뷰 API 호출) 로 로그인 필요 여부 확인 | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이 부분의 토큰 절차를 어디서 처리할지 애매하네요..
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아니면 스플래시에서 별도의 체킹 없이
토큰 인터셉터에서 리프레쉬 만료 관련 에러가 뜨면 로그인으로 가게 하면 될까요.?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
월요일 회의때 더 자세히 설명해주시면 감사하겠습니다! 고생했어요!
👾 작업 내용
화면 전환을 담당하는 코디네이터를 설계했습니다.
🚀 PR Point
(⭐많은 피드백⭐ 부탁드립니다..)
코디네이터 설계
초반 설계
코디네이터 모듈을 따로 정의, RootFeature에서 View, ViewModel을 주입하려 했음
Flow, Action을 정의하여 Combine으로 그 상태를 관리하려 했음
과연 코디네이터가 필요할까?
완성된 구조
ex.
챌린지 뷰에서 포인트뷰로 이동 )는 NavigationPath에 append 하는 방식으로 구현ex.
로그인 이후 상태를 온보딩 또는 탭바로 이동 ) 추후에는 탭바 내에서 다른 탭 아이템으로 이동해야할 때 사용.✅ CheckList
🔗 Issue
Resolved #104