[#578] TodayView에 TCA를 적용한다#606
Conversation
There was a problem hiding this comment.
Code Review
이번 PR은 기존 MVVM/Coordinator 패턴의 Today 화면을 Composable Architecture(TCA) 기반의 TodayFeature로 마이그레이션하고 관련 뷰 및 테스트 코드를 추가한 변경사항을 담고 있습니다. 코드 리뷰에서는 시간 민감한 연산의 일관성을 유지하고 Date() 생성 오버헤드를 줄이기 위해, sections 및 summaryCounts 진입점에서 현재 시간을 한 번만 캡처하여 isOverdue, isDueSoon, summaryValue, groupedSectionItems 등의 하위 메서드에 매개변수로 주입하도록 개선할 것을 권장하고 있습니다.
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.
|
반영 상태 09b5009 기존 구현은 함께 반영한 범위는 아래와 같음.
이번 수정으로 시간 의존 헬퍼의 순수성이 조금 더 명확해졌고 자정 경계 같은 희소 케이스에 대해서도 계산 기준 시각이 한 번으로 맞춰진 상태 |
🔗 연관된 이슈
🎯 의도
📝 작업 내용
📌 요약
@Bindable기준으로 정리한 작업BindingAction기반으로 정리한 작업🔍 상세
todos,selectedSectionScope,displayOptions,loading,alert상태를 TCAState로 이관한 작업fetchData,setSectionScope, display options 변경,togglePinned,completeTodo, 에러 표시 상태에 대한 패리티 테스트 추가 작업@Bindable var store를 사용하도록 정리한 작업BindingAction경로로 연결한 작업setDueDateVisibility,setFocusVisibility중복 액션 제거 후 store 테스트 어댑터도 binding 전송 기반으로 정리한 작업@Bindable로 정리한 작업📸 영상 / 이미지 (Optional)