Skip to content

[#573] SettingsView에 TCA를 적용한다#591

Merged
opficdev merged 5 commits into
developfrom
refactor/#SettingsView-TCA
Jun 12, 2026
Merged

[#573] SettingsView에 TCA를 적용한다#591
opficdev merged 5 commits into
developfrom
refactor/#SettingsView-TCA

Conversation

@opficdev

@opficdev opficdev commented Jun 12, 2026

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🔗 연관된 이슈

🎯 의도

  • SettingsView의 ViewModel 기반 상태 관리를 TCA Feature/Store 기반으로 전환

📝 작업 내용

📌 요약

  • SettingsViewModel 제거 및 SettingsFeature 추가
  • SettingsView, ThemeView, ProfileViewCoordinator, MainView, ProfileView의 Store/Binding 연결 반영
  • SettingsFeature 상태 변화와 effect 흐름을 검증하는 TestStore 기반 테스트 추가

🔍 상세

  • 기존 SettingsViewModel의 network observing, theme monitoring, cache size 조회, cache 삭제, sign out, delete auth, alert, loading 흐름을 SettingsFeature reducer/effect로 이전
  • ThemeView는 store 주입 없이 BindingAction 기반 $settingsStore.theme 연결로 유지
  • Settings 관찰 시작 중복 전송 방지를 위한 state guard 반영
  • alert button action 경로를 포함한 SettingsFeatureTests 추가
  • SettingsFeatureTestDoubles로 use case spy 분리

📸 영상 / 이미지 (Optional)

@opficdev opficdev self-assigned this Jun 12, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

이번 풀 리퀘스트는 기존의 SettingsViewModel을 Composable Architecture(TCA) 기반의 SettingsFeature로 리팩터링하고 관련 뷰와 테스트 코드를 업데이트합니다. 리뷰 피드백으로는 SettingsFeature 내에서 실제 사용되지 않는 중복 액션 및 분기 처리를 제거하여 코드를 정리할 것과, 불필요한 리소스 소모를 방지하기 위해 네트워크 및 테마 관찰 시작 시점을 코디네이터 초기화 단계에서 SettingsViewonAppear 시점으로 이동할 것을 제안했습니다.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread Application/DevLogPresentation/Sources/Settings/SettingsFeature.swift Outdated
Comment thread Application/DevLogPresentation/Sources/Settings/SettingsView.swift
@opficdev opficdev merged commit 5889f7b into develop Jun 12, 2026
5 checks passed
@opficdev opficdev deleted the refactor/#SettingsView-TCA branch June 12, 2026 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SettingsView에 TCA를 적용한다

1 participant