Skip to content

fix: 디데이 설정 화면에서의 흐름제어 변경#297

Merged
clxxrlove merged 4 commits into
developfrom
fix/#289/TWI-79
May 14, 2026
Merged

fix: 디데이 설정 화면에서의 흐름제어 변경#297
clxxrlove merged 4 commits into
developfrom
fix/#289/TWI-79

Conversation

@clxxrlove
Copy link
Copy Markdown
Member

@clxxrlove clxxrlove commented May 13, 2026

🔗 관련 이슈

📙 작업 내역

💬 추가 설명 or 리뷰 포인트 (선택)

  • 지금 서버에서 상대가 디데이 등록 마쳐도 온보딩 상태가 변경이 안되고 있는듯 그래서 동작 테스트가 안됨
  • 추가로 초대장 주소 변경은 사소해서 온보딩 건드는 겸에 껴넣었음

@linear
Copy link
Copy Markdown

linear Bot commented May 13, 2026

TWI-79

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Review Change Stack

Important

Review skipped

Auto reviews are disabled on this repository. 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: CHILL

Plan: Pro Plus

Run ID: dcdbd248-1153-41e9-b377-7a00c4391a2a

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
📝 Walkthrough

Walkthrough

Dday 기념일 설정 화면이 파트너의 실시간 상태를 지속적으로 확인하도록 수정되었습니다. 화면 진입 시 3초 간격의 폴링을 시작하고, 파트너가 기념일을 완료하면 모달로 알리며 중복 제출을 방지합니다. 사용자가 기념일을 설정하거나 화면을 벗어날 때 폴링을 정리합니다.

Changes

Dday 파트너 상태 지속 폴링

Layer / File(s) Summary
폴링 상태 및 액션 정의
Projects/Feature/Onboarding/Sources/Dday/OnboardingDdayReducer.swift
Reducer에 @Dependency(\.continuousClock)으로 지속적 시계를 추가하고, modal 상태 필드를 도입합니다. Action enum에 .onAppear, .pollingTick, .pollingResult, .modalConfirmTapped 액션 케이스를 추가합니다.
폴링 및 기념일 완료 흐름 구현
Projects/Feature/Onboarding/Sources/Dday/OnboardingDdayReducer.swift
.onAppear 처리에서 3초 간격의 반복 타이머를 시작하여 .pollingTick을 트리거합니다. .completeButtonTapped는 폴링을 취소하고 기념일 설정 요청을 병합 효과로 수행합니다. .setAnniversaryResponse, .pollingResult, .modalConfirmTapped를 처리하여 파트너 완료 감지 시 모달을 표시하고 흐름을 위임합니다.
뷰 통합 및 좌표 정리
Projects/Feature/Onboarding/Sources/Dday/OnboardingDdayView.swift, Projects/Feature/Onboarding/Sources/OnboardingCoordinator.swift
View의 .body.onAppear { store.send(.onAppear) }와 모달 바인딩 .txModal(item: $store.modal)을 추가합니다. Coordinator에서 Dday 화면 이탈 시 OnboardingDdayReducer.CancelID.polling을 취소하도록 변경합니다.

Sequence Diagram(s)

sequenceDiagram
  participant View as OnboardingDdayView
  participant Reducer as OnboardingDdayReducer
  participant Client as onboardingClient
  
  View->>Reducer: .onAppear
  Reducer->>Reducer: 3초 간격 .pollingTick 타이머 시작
  
  loop 폴링 반복
    Reducer->>Client: fetchOnboardingStatus()
    Client-->>Reducer: .pollingResult
    alt 파트너 완료됨
      Reducer->>Reducer: 모달 표시, 폴링 취소
    else 아직 진행 중
      Reducer->>Reducer: 폴링 계속
    end
  end
  
  View->>Reducer: .completeButtonTapped
  Reducer->>Reducer: 폴링 취소
  Reducer->>Client: setAnniversary(date)
  Client-->>Reducer: .setAnniversaryResponse
  alt 성공
    Reducer->>View: .ddayCompleted 위임
  else 이미 온보딩됨
    Reducer->>Reducer: 모달 표시
  end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related issues

Suggested labels

🐞 fix

Suggested reviewers

  • jihun32
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#289/TWI-79

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

@clxxrlove
Copy link
Copy Markdown
Member Author

@coderabbitai summary

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

✅ Actions performed

Summary regeneration triggered.

@clxxrlove clxxrlove marked this pull request as draft May 13, 2026 11:47
@clxxrlove
Copy link
Copy Markdown
Member Author

다른작업 추가해야해서 일단 얼려둠

@clxxrlove clxxrlove marked this pull request as ready for review May 13, 2026 12:44
Copy link
Copy Markdown
Contributor

@jihun32 jihun32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오오 continuousClock 이런것도 있네 배웠다 ㅋㅋ
고생했어~

@clxxrlove clxxrlove merged commit 7168f5e into develop May 14, 2026
4 checks passed
@clxxrlove clxxrlove deleted the fix/#289/TWI-79 branch May 14, 2026 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants