Skip to content

(REFACTOR)[#554] BaseViewModel 적용을 통한, 'Navigator'와 'errorFlow' 공통 처리 #555

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.

Sign up for GitHub

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

Jump to bottom
Open
squart300kg wants to merge 17 commits into
base: main
Choose a base branch
from
Open

(REFACTOR)[#554] BaseViewModel 적용을 통한, 'Navigator'와 'errorFlow' 공통 처리 #555

squart300kg wants to merge 17 commits into from

Conversation

@squart300kg
Copy link

@squart300kg squart300kg commented Aug 22, 2025

Issue

Overview (Required)

  • 현재 대부분의 ViewModel(SessionDetail, SessionList, Home, Bookmark, Contributor, Main)에선 'Navigator'를 활용한 화면 이동 로직이 정의되어 있습니다.
  • 현재 대부분의 ViewModel에선 '_errorFlow = MutableSharedFlow()'를 활용한 에러처리 플로우가 정의되어 있습니다.
  • 따라서 위 2가지('_errorFlow', 'Navigator')를 BaseViewModel로의 정의 및 공통 로직으로 관리하고자 합니다.
  • 기대효과로, 기존 ViewModel의 중복로직을 크게 감소시킬 수 있습니다. (eg., MainViewModel의 라인 수 50 -> 15 감소)

Verification

  • BaseViewModel의 신규 정의 및 기존 *.ViewModel들에 상속 처리로 인해, *.ViewModelTest 검증 완료.

@squart300kg
(CHORE)[droidknights#554] 'core:ui' 'core.router.routerApi' 의존성 추가
32cadc2
@squart300kg
(FEAT)[droidknights#554] BaseViewModel.kt 정의
6cf9af9
@squart300kg
(REFACTOR)[droidknights#554] MainViewModel.kt에 BaseViewModel.kt 상속 처리…
abb6816 
… 후 코드 삭제. 그에따른 MainActivity.kt의 화면 이동코드 수정
@squart300kg
(FIX)[droidknights#554] BaseViewModel내, Navigator 테스트 가능하도록 수정
0b679d5
@squart300kg
(TEST)[droidknights#554] MainViewModelTest 수정
89ff34c
@squart300kg
(REFACTOR)[droidknights#554] BookmarkViewModel.kt에 BaseViewModel.kt 상…
457345f 
…속 처리 후 코드 삭제.
@squart300kg
(TEST)[droidknights#554] BookmarkViewModelTest.kt 수정
4c04ebe
@squart300kg
(REFACTOR)[droidknights#554] SessionDetailViewModel.kt에 BaseViewModel…
0eb4165 
….kt 상속 처리 후 코드 삭제.
@squart300kg
(TEST)[droidknights#554] SessionDetailViewModelTest.kt 수정
0b6b2a8
@squart300kg
(REFACTOR)[droidknights#554] SessionListViewModel.kt에 BaseViewModel.k…
deeb8b8 
…t 상속 처리 후 코드 삭제.
@squart300kg
(TEST)[droidknights#554] SessionListViewModelTest.kt 수정
89b944d
@squart300kg
(REFACTOR)[droidknights#554] HomeViewModel.kt에 BaseViewModel.kt 상속 처리…
75db57b 
… 후 코드 삭제. 그에따른 HomeScreen.kt의 화면 이동코드 수정
@squart300kg
(TEST)[droidknights#554] HomeViewModelTest.kt 수정
1c66255
@squart300kg
(REFACTOR)[droidknights#554] ContributorViewModel.kt에 BaseViewModel.k…
08cb42b 
…t 상속 처리 후 코드 삭제.
@squart300kg
(TEST)[droidknights#554] ContributorViewModelTest.kt 수정
980f8fa
@squart300kg
(STYLE)[droidknights#554] detekt 적용
4f0277b
@squart300kg
(STYLE)[droidknights#554] detekt 적용을 위한 백킹 프로퍼티 네이밍 변경(_errorFlow -> …
533abc7 
…internalFlow)
@github-actions
Copy link

github-actions bot commented Aug 22, 2025

Test Results

61 tests   61 ✅  27s ⏱️
28 suites   0 💤
28 files     0 ❌

Results for commit 533abc7.

@squart300kg
Copy link
Author

squart300kg commented Aug 22, 2025

@taehwandev
안녕하세요, 권태환님. 드로이드나이츠 연사로 발표했던 송상윤입니다.

늦었지만 드로이드나이츠 코드를 공부하고, 컨트리뷰터로도 참여해보고자 PR을 올렸습니다.
누구에게 요청드려야 할지 몰라 권태환님께 리뷰를 부탁드리려 하는데요. 혹시 리뷰가 가능할까요? 혹시나마 제가 절차적으로 미준수한 게 있다면 말씀해주세요!

감사하겠습니다! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[App] BaseViewModel 적용을 통한, 'Navigator'와 'errorFlow' 공통 처리

1 participant

@squart300kg