Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 Pull Request에서는 MainView의 복잡한 상태 관리와 ViewModel 생성을 전담하는 MainViewCoordinator를 도입하여 관심사를 분리하였습니다. 이에 따라 MainView 내부의 여러 NavigationRouter와 ViewModel 생성 로직이 Coordinator로 이동되었습니다. 리뷰에서는 MainView에 남아있는 DIContainer 의존성을 완전히 제거하기 위해 팩토리 메서드들을 Coordinator로 옮길 것과, TodoDetailViewModel 생성 시 편집 버튼 노출 여부를 제어할 수 있는 파라미터를 추가할 것을 제안하였습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 연관된 이슈
📝 작업 내용
📌 요약
MainViewCoordinator도입으로MainView의 루트 상태 및 라우터 소유 책임 분리MainView를UI/Common에서UI/Main디렉토리로 이동MainView내부 바인딩/헬퍼를 extension으로 정리🔍 상세
MainViewCoordinator에서MainViewModel, 탭별 루트 ViewModel,HomeRoute/TodayRoute라우터, 알림 상세 선택 상태 관리RootView에서MainViewModel을 직접 생성하지 않고DIContainer를MainView에 전달하도록 수정MainView의 루트 UI 구성은 유지하면서 ViewModel 및 Router 접근 경로를 coordinator 기준으로 정리MainView의 바인딩 생성 로직을 private extension으로 분리해 body 주변 책임 축소📸 영상 / 이미지 (Optional)