Skip to content

[fix] #627 홈 TODAY'S QUESTION 답변 CTA 를 작성 화면에 연결한다 - #894

Merged
Sadturtleman merged 2 commits into
developfrom
fix/627-today-question-cta
Aug 24, 2026
Merged

Sadturtleman merged 2 commits into
developfrom
fix/627-today-question-cta

Conversation

@Sadturtleman

Copy link
Copy Markdown
Collaborator

📌𝘐𝘴𝘴𝘶𝘦𝘴

Closes #627

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

배선은 카드부터 끝까지 이미 있었고, 마지막 구현만 빈 TODO 였습니다.

// AppNavigationActions.kt (수정 전)
override fun onAnswerClick() {
    // TODO: 데일리 질문 답변 화면 Route 추가 후 연결
}

이슈가 적은 "카드 데이터는 정상인데 CTA 만 죽어 있다" 와 정확히 일치합니다. MindRecordRoute.DailyQuestionWriteRoute 는 이미 존재하고 루트 NavHost 에 등록돼 있어(AppNavigation.kt:129) 홈에서도 그대로 도달합니다.

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵 · 실측

에뮬레이터(Medium_Phone / API 36) debug 빌드, 실계정 로그인 상태에서 확인했습니다.

수정 전 — "데일리질문 답변하기" 탭 후 화면 텍스트가 그대로입니다.

안녕하세요, adamtia님 | TODAY'S QUESTION | 2026.08.23 |
사랑하는 사람에게 듣고 싶은 칭찬 한마디는? | 데일리질문 답변하기 | ...

logcat OkHttp 호출 0건 — 이슈 본문의 실측과 같습니다.

수정 후 — 같은 질문의 답변 작성 화면으로 이동합니다.

Day 14 | 오늘의 추천 질문 | 사랑하는 사람에게 듣고 싶은 칭찬 한마디는? |
당신의 오늘을 기록해보세요 | 0 글자 | 임시저장 | 2026.08.23 | 저장

카드에 떠 있던 질문과 작성 화면의 질문이 같은 것도 확인했습니다.

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

  • 검증: :app:assembleDebug + :app:ktlintCheck BUILD SUCCESSFUL, 위 에뮬 실측.
  • ⚠️ 목적지는 시안 확정 근거 없이 정했습니다. 이슈 본문도 "이 CTA 의 목적지(데일리질문 답변 작성 화면인지 기록 탭인지)가 시안에서 어떻게 정의돼 있는지는 확인하지 않았다" 로 남겨 뒀습니다. 카드 문구가 "데일리질문 답변하기" 이고 같은 동작의 기존 진입점이 작성 화면이라 그쪽으로 잡았습니다. 시안이 기록 탭 이동으로 정의돼 있다면 이 한 줄만 바꾸면 됩니다.
  • 이 PR 은 추측으로 정한 항목이라 docs/qa/assumptions.md 에도 함께 기록합니다.

카드 → `TodayQuestionCard.onAnswerClick` → `HomeTabActions.onAnswerClick` 까지
배선은 이미 있었고, 마지막 구현만 빈 TODO 로 남아 있었다. 그래서 탭해도 화면 전환도
네트워크 호출도 없었다.

`MindRecordRoute.DailyQuestionWriteRoute` 로 보낸다 — 기록 탭의 작성 진입
(`MindRecordNavActions.onWriteDailyQuestion`)과 같은 목적지이고, 그래프가 루트
NavHost 에 등록돼 있어 홈에서도 그대로 도달한다.

Closes #627

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@OnlyReadOneBook OnlyReadOneBook left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

승인합니다. 비차단 1건 참고해 주세요.

  1. [normal] 본문이 말한 assumptions.md 기록은 이 PR이 아니라 #897에 있습니다.
    • 본문의 "docs/qa/assumptions.md 에도 함께 기록합니다"가 이 PR diff에는 없고, 실제 기록(#627 행 — 목적지 추측과 뒤집을 자리)은 #897에 들어 있습니다. 본문에 #897 링크를 걸어 두면 추적이 됩니다.

검증:

  • CI 전 체크 green (1-hunk 변경, 컴파일 입증)
  • 홈 CTA → 이 액션의 호출 경로 전수 추적: AppNavigation.kt:97 composable(Home) → rememberHomeTabActions → HomeTabScreen:253 onAnswerClick = actions::onAnswerClick → HomeTabMindRecordLazyItems.kt:48-52 → TodayQuestionCard.kt:144 onClick — 실제 결선 확인, 다른 구현은 프리뷰 Noop뿐
  • 목적지 도달성: DailyQuestionWriteRoute가 같은 루트 NavHost의 mindRecordNavGraph에 등록, 기록 탭 진입과 동일 패턴
  • 질문 id·날짜 전달 불요 확인 — 목적지 VM이 getToday() 자체 조회, 홈 요약과 동일 소스
  • 이미 답변한 경우: draft만 프리필(draftOnly=true)이라 확정 답변은 빈 에디터로 열리고 저장 시 서버 upsert로 갱신 — 기록 탭 기존 진입점과 완전히 동일한 동작이므로 이 PR의 결함 아님. 목적지 선택은 이슈·#897 기록대로 FE 재량 범위로 판단

@Sadturtleman
Sadturtleman merged commit 0b3b75c into develop Aug 24, 2026
14 checks passed
@Sadturtleman
Sadturtleman deleted the fix/627-today-question-cta branch August 24, 2026 05:54
@github-actions

Copy link
Copy Markdown
Contributor

🚀 QA 배포 권장

QA 포인트

이 판단은 배포를 자동 실행하지 않습니다. deploy일 때 Release Distribution을 수동 실행하세요.

This was referenced Aug 24, 2026
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.

fix(mindrecord): 홈 TODAY'S QUESTION 카드 '데일리질문 답변하기' 무동작 — 카드 데이터는 정상

2 participants