[#356] 차트를 제거하고 히트맵 UI를 구체화하며 Todo 삭제도 집계되도록 개선한다#359
Conversation
opficdev
commented
Apr 5, 2026
opficdev
commented
- closed 차트를 제거하고 히트맵 UI를 구체화하며 Todo 삭제도 집계되도록 개선한다 #356
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request transitions the todo deletion logic from a boolean flag to a timestamp-based soft delete (deletedAt) and introduces a compaction strategy for old deleted items. It also enhances the profile activity heatmap by including deleted activities and refactoring the underlying data structures. Feedback focuses on a potential issue with the Firestore query for non-compacted documents, the presence of unused helper methods in ProfileViewModel, and suggestions to optimize the parallel fetching of activity data to reduce Firestore costs.
| let query = admin.firestore() | ||
| .collectionGroup("todoLists") | ||
| .where("isDeleted", "==", true) | ||
| .where("compactedAt", "==", null) |
There was a problem hiding this comment.
Firestore에서 필드가 존재하지 않는 문서는 해당 필드에 대한 쿼리 결과에 포함되지 않습니다. requestTodoDeletion에서 compactedAt 필드를 null로 초기화하지 않으므로, where("compactedAt", "==", null) 쿼리는 압축되지 않은 문서를 찾지 못할 가능성이 큽니다. isCompacted와 같은 불리언 플래그를 사용하거나, deletedAt 범위 쿼리 후 코드 레벨에서 필터링하는 방안을 검토해 주세요.
| func dayKey(from date: Date) -> String { | ||
| date.formatted( | ||
| Date.ISO8601FormatStyle(timeZone: calendar.timeZone) | ||
| .year() | ||
| .month() | ||
| .day() | ||
| ) | ||
| } | ||
|
|
||
| func date(from dayKey: String) -> Date? { | ||
| try? Date( | ||
| dayKey, | ||
| strategy: Date.ISO8601FormatStyle(timeZone: calendar.timeZone) | ||
| .year() | ||
| .month() | ||
| .day() | ||
| ) | ||
| } |
There was a problem hiding this comment.
dayKey(from:)와 date(from:) 메서드는 현재 클래스 내에서 사용되지 않는 것으로 보입니다. 불필요한 코드는 제거하여 유지보수성을 높이는 것이 좋습니다.
| async let createdTodoPage = fetchTodosUseCase.execute( | ||
| TodoQuery( | ||
| createdAtFrom: quarterStart, | ||
| createdAtTo: nextQuarterStart, | ||
| sortDateFrom: quarterStart, | ||
| sortDateTo: nextQuarterStart, | ||
| includesDeleted: true, | ||
| sortTarget: .createdAt, | ||
| pageSize: 100, | ||
| fetchAllPages: true | ||
| ), | ||
| cursor: nil | ||
| ) | ||
| return page.items | ||
| } | ||
|
|
||
| func fetchEarliestQuarterStart() async throws -> Date { | ||
| let page = try await fetchTodosUseCase.execute( | ||
| async let completedTodoPage = fetchTodosUseCase.execute( | ||
| TodoQuery( | ||
| sortTarget: .createdAt, | ||
| sortOrder: .oldest, | ||
| pageSize: 1 | ||
| sortDateFrom: quarterStart, | ||
| sortDateTo: nextQuarterStart, | ||
| includesDeleted: true, | ||
| sortTarget: .completedAt, | ||
| pageSize: 100, | ||
| fetchAllPages: true | ||
| ), | ||
| cursor: nil | ||
| ) | ||
| async let deletedTodoPage = fetchTodosUseCase.execute( | ||
| TodoQuery( | ||
| sortDateFrom: quarterStart, | ||
| sortDateTo: nextQuarterStart, | ||
| includesDeleted: true, | ||
| sortTarget: .deletedAt, | ||
| pageSize: 100, | ||
| fetchAllPages: true | ||
| ), | ||
| cursor: nil | ||
| ) |
There was a problem hiding this comment.
분기 활동 데이터를 가져오기 위해 세 개의 독립적인 쿼리를 병렬로 실행하고 있습니다. fetchAllPages: true 설정으로 인해 데이터 양이 많을 경우 Firestore 읽기 비용과 네트워크 부하가 증가할 수 있습니다. 활동 요약 데이터를 서버 측에서 미리 집계하거나, 클라이언트 측 캐싱 전략을 강화하는 것을 고려해 보세요.
1 participant
