Skip to content

[#577] PushNotificationListView에 TCA를 적용한다#605

Merged
opficdev merged 13 commits into
developfrom
refactor/#577-PushNotificationListView-TCA
Jun 14, 2026
Merged

[#577] PushNotificationListView에 TCA를 적용한다#605
opficdev merged 13 commits into
developfrom
refactor/#577-PushNotificationListView-TCA

Conversation

@opficdev

Copy link
Copy Markdown
Owner

🔗 연관된 이슈

🎯 의도

  • PushNotificationListView의 TCA 이행 이후 남아 있던 화면 상태 관리와 UI 구조 차이를 정리
  • compact / regular 전환, 필터 헤더, 삭제 토스트 처리 흐름을 기존 의도에 맞게 안정화

📝 작업 내용

📌 요약

  • PushNotificationListView의 시트 동기화 방식 정리
  • PushNotificationListView의 헤더/리스트 구조 정리
  • PushNotificationListFeature의 액션 구조를 외부 입력과 내부 응답 관점에서 일부 정리
  • TodoListFeature, SearchFeature도 동일 기준으로 내부 응답 액션 분리 적용

🔍 상세

  • compact 레이아웃에서만 시트가 표시되고 regular 전환 시 즉시 내려가도록 조정
  • regular에서 선택 상태는 유지하되 compact 복귀 시 다시 시트가 올라오도록 동기화 로직 정리
  • PushNotificationListView의 빈 상태 처리 방식을 Text 분기에서 List 내부 row 방식으로 변경하여 헤더 재마운트에 따른 깜빡임 완화
  • PushNotificationListView의 헤더 구성을 정리하여 불필요한 컴포넌트 레이어 제거
  • PushNotificationList의 필터 PickerTodoListFeature와 동일하게 root binding 방식으로 정리
  • PushNotificationList의 삭제 토스트 트리거를 실제 삭제 진입점인 swipe action 기준으로 단순화
  • PushNotificationListFeature는 View에서 보내는 top-level 액션은 유지하고, effect 내부 응답만 store(StoreAction)로 분리
  • 같은 기준을 TodoListFeature, SearchFeature에도 적용하여 effect 응답 액션을 store(StoreAction)로 분리
  • 관련 테스트 더블도 새 액션 구조에 맞게 정리

📸 영상 / 이미지 (Optional)

  • 없음

@opficdev opficdev self-assigned this Jun 14, 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

이번 PR은 푸시 알림 목록 화면을 TCA(The Composable Architecture) 기반의 PushNotificationListFeature로 마이그레이션하고, 기존 TodoListFeatureSearchFeature 액션들을 StoreAction으로 그룹화하는 리팩토링을 포함합니다. 리뷰 피드백으로는 알림 목록이 비어 있을 때 List 내부 분기 대신 비스크롤형 뷰를 사용하여 빈 화면을 처리할 것과, toggleRead 액션의 낙관적 업데이트 실패 시 상태를 되돌릴 수 있는 롤백 로직을 추가할 것을 제안합니다.

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.

@opficdev opficdev merged commit 9e1c5c3 into develop Jun 14, 2026
5 checks passed
@opficdev opficdev deleted the refactor/#577-PushNotificationListView-TCA branch June 14, 2026 05:42
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.

PushNotificationListView에 TCA를 적용한다

1 participant