Skip to content

[Feat] [행운액션] 행운 액션 화면 및 완료 흐름 구현 (#95) - #96

Merged
seungwone merged 6 commits into
releasefrom
feature/95-lucky-action
Aug 16, 2026
Merged

seungwone merged 6 commits into
releasefrom
feature/95-lucky-action

Conversation

@seungwone

@seungwone seungwone commented Aug 15, 2026 •

Copy link
Copy Markdown
Collaborator

변경 내용

  • 당일/과거 행운 액션 조회, 날짜 이동, 액션 체크·점수 반영을 구현했습니다.
  • 행운 액션 탭 진입과 Fortune 상세 리포트 push 진입을 분리했습니다.
  • 완료 오버레이를 App 전역 host로 조립해 본문·탭바·safe area가 단일 딤/애니메이션으로 전환되도록 했습니다.
  • 완료 말풍선·캐릭터·클로버·날짜 화살표 리소스를 반영하고 Example 완료 시나리오도 구성했습니다.
  • 실패한 과거 일자 조회에서도 날짜 이동·재시도가 가능하도록 보완하고, 완료 안내 큐의 자동 종료 취소·재시작을 테스트로 고정했습니다.

고민한 내용

  • 완료 상태·타이머·큐는 LuckyActionFeature가 소유하고, App은 전역 z-index와 딤만 소유하도록 분리했습니다.
  • 이 구조로 탭바 딤과 본문 딤이 서로 다른 뷰 트리에서 전환되며 발생하던 시차를 제거했습니다.

스크린샷

  • 요청에 따라 첨부를 생략했습니다.

테스트

  • iPhone SE (3rd generation, iOS 17.5) Todakun 전체 테스트
  • LuckyActionFeatureExample 빌드
  • SwiftLint
  • ./scripts/sync-and-validate.sh

PR Preflight Review

  • Status: 완료
  • Mode: Quick
  • Roles: Architecture Guard, TCA/Regression, Build/Tooling & CI
  • Rounds: 2회 (초기 검토, 반영 후 Sol 재검토)
  • Accepted findings: 과거 조회 실패 화면의 날짜 이동, 완료 큐 자동 종료 취소·재시작, Fortune → LuckyAction push와 back 경로, 클라이언트 오류 매핑 테스트를 보완했습니다.
  • Rejected findings: Interface 공개 타입 범위 축소는 App 전역 오버레이와 Testing에서 공유 모델이 필요한 현재 경계에 비해 이득이 작아 별도 리팩터링으로 보류했습니다.
  • Verification: 전체 테스트, Example 빌드, SwiftLint, 아키텍처 검증 및 pre-push hook 통과
  • Residual risk: 실패 화면의 작은 기기 실제 레이아웃은 수동 UI QA가 남아 있습니다. 스크린샷은 요청에 따라 생략했습니다.

참고 사항

  • 06시 기준 액션 일자는 현재 KST 기기 시간으로 계산하며, 자동 갱신 타이머는 두지 않았습니다.
  • 토닥이 채팅 고정 답변 타이핑 연동은 서버/Feature 준비 후 별도 작업입니다.

체크리스트

  • 로컬에서 정상 동작을 확인했습니다.
  • 필요한 테스트를 실행했습니다.
  • PR Preflight Review를 수행했습니다.
  • UI 변경 스크린샷은 요청에 따라 생략했습니다.
  • 문서나 설정 변경이 필요한지 확인했습니다.

Closes #95

- 행운 액션 Interface·Live Client·Testing fixture와 Example을 구성하고 당일 및 날짜별 조회·달성 상태 변경을 연결한다.
- 탭과 Fortune 상세 push 진입을 App에서 조립하고, 날짜 경계·완료 안내·과거 읽기 전용 흐름을 TCA 상태와 테스트로 보장한다.
- 완료 에셋·날짜 아이콘·공통 pressed overlay를 적용하고 아키텍처 검증 규칙과 의존성 다이어그램을 현재 Interface 조립 구조에 맞춘다.
- 기획에 없는 Fortune 홈 pull-to-refresh UI와 refresh 상태·액션을 제거한다.
- 화면 이탈 시 진행 중인 운세 조회를 취소하는 네트워크 수명 정리 로직은 유지한다.
- Gender와 BirthDateCalendar의 공통 title API 변경에 맞춰 DesignSystem 명세 테스트를 갱신한다.
- MyPage 프로필 fixture를 현재 서버 enum 값과 동일하게 맞춰 전체 테스트 회귀를 방지한다.
- Feature의 완료 상태와 콘텐츠를 유지한 채 App이 단일 전역 딤과 z-index를 조립한다.
- 탭과 push 화면의 완료 상태·닫기 액션을 올바른 하위 Feature로 연결한다.
- Example에서도 완료 오버레이 host를 조립해 완료 시나리오를 유지한다.
- 완료 말풍선의 이모지를 기획 명세에 맞춰 💌으로 변경한다.
@seungwone seungwone self-assigned this Aug 15, 2026
@github-actions github-actions Bot added the enhancement 기능 개발, 리팩토링, 설정 변경 등 전반적인 작업 label Aug 15, 2026
@coderabbitai

coderabbitai Bot commented Aug 15, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 74629372-caa0-40d0-8a32-ccc244421e75

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@seungwone
seungwone changed the base branch from main to release August 15, 2026 17:15
- 과거 날짜 조회 실패 상태에서도 날짜 이동과 재시도를 유지합니다.\n- 완료 안내 큐의 자동 종료 취소·재시작을 검증합니다.\n- 클라이언트 오류와 리포트 전환 회귀 테스트를 추가합니다.
@seungwone seungwone changed the title [Feat] [행운액션] 당일 행운 액션 화면 및 완료 흐름 구현 (#95) [Feat] [행운액션] 행운 액션 화면 및 완료 흐름 구현 (#95) Aug 16, 2026
@seungwone
seungwone marked this pull request as ready for review August 16, 2026 04:24
@seungwone
seungwone requested a review from MoonGoon72 as a code owner August 16, 2026 04:24
@seungwone
seungwone merged commit 71a317e into release Aug 16, 2026
5 checks passed
@seungwone
seungwone deleted the feature/95-lucky-action branch August 16, 2026 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 기능 개발, 리팩토링, 설정 변경 등 전반적인 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[iOS] [Feat] [행운액션] 당일 행운 액션 화면 및 완료 흐름 구현

1 participant