[#384] WidgetKit 기반 Today/Heatmap 위젯의 데이터 소스를 구현한다#394
Merged
Conversation
Contributor
There was a problem hiding this comment.
Code Review
이번 PR은 TodayTodoWidget 및 HeatmapWidget 기능을 추가하고, 이를 위해 ViewModel 내 동기화 로직과 App Group 기반의 공유 저장소를 구현했습니다. 주요 피드백으로는 리듀서(reduce) 내에서 직접 비동기 작업을 생성하거나 동기 I/O를 수행하는 아키텍처 위반 사항이 발견되어 SideEffect로의 이관이 필요합니다. 또한 .pbxproj 내 비정상적인 버전 번호 수정, 타겟 간 중복 코드 공유 설정, 그리고 동기화 과정의 에러 핸들링 보완을 권장합니다.
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.
🔗 연관된 이슈
📝 작업 내용
📌 요약
🔍 상세
WidgetSharedDefaultsStore,WidgetSnapshotStore구성TodayWidgetSnapshotFactory추가TodayWidgetSyncCoordinator추가TodayViewModel상태 변경 시 Today 위젯 스냅샷 저장 및 리로드 연결HeatmapWidgetSnapshotFactory추가HeatmapWidgetSyncCoordinator추가ProfileViewModel에서 현재 월 기준 Heatmap 위젯 스냅샷 저장 및 리로드 연결Profile접두사 제거 및 위젯 지원 레이어 네이밍 정리📸 영상 / 이미지 (Optional)