Skip to content

[Fix]: 운세 상세 바텀시트 '토닥이에게 더 물어보기' 라우팅 연결 및 뒤로가기 재노출 방지 - #162

Open
oungsi2000 wants to merge 1 commit into
developfrom
fix/chat-cta-routing
Open

[Fix]: 운세 상세 바텀시트 '토닥이에게 더 물어보기' 라우팅 연결 및 뒤로가기 재노출 방지#162
oungsi2000 wants to merge 1 commit into
developfrom
fix/chat-cta-routing

Conversation

@oungsi2000

Copy link
Copy Markdown
Collaborator

관련 이슈

close #38

작업 내용

홈/운세 리포트 화면의 운세 상세 바텀시트에 있는 '토닥이에게 더 물어보기' 버튼에 라우팅이 연결되어 있지 않던 문제 수정

변경사항 / 상세

  • FortuneDetailBottomSheet의 CTA 버튼이 TODO(#38)로 비어있어 클릭해도 아무 동작이 없던 문제를 채팅 화면(TodakunRoute.Chat) 라우팅으로 연결
    • onAskTodakClick 콜백을 추가해 HomeScreen/FortuneReportScreenHomeRoute/FortuneReportRouteTodakunApp까지 관통
  • 바텀시트가 열린 채로 채팅에 진입한 뒤 채팅에서 뒤로가기를 누르면, 백스택에 남아있던 detail 상태 때문에 바텀시트가 다시 열리는 버그가 있어 채팅 이동 전 closeDetail()로 상태를 먼저 비우도록 수정

중점 리뷰사항

  • 홈 화면과 운세 리포트 화면 양쪽에서 바텀시트를 연 뒤 '토닥이에게 더 물어보기' → 채팅 → 뒤로가기 순서로 눌렀을 때 바텀시트가 다시 뜨지 않는지 확인

스크린샷 (선택)

🤖 Generated with Claude Code

@coderabbitai

coderabbitai Bot commented Aug 24, 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: CHILL

Plan: Pro Plus

Run ID: 73256e47-8879-4caf-8655-8aca26233c3b

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.

@oungsi2000 oungsi2000 self-assigned this Aug 24, 2026
@oungsi2000 oungsi2000 added the fix 버그 수정 label Aug 24, 2026
@oungsi2000 oungsi2000 changed the title fix: 운세 상세 바텀시트 '토닥이에게 더 물어보기' 라우팅 연결 및 뒤로가기 재노출 방지 [Fix]: 운세 상세 바텀시트 '토닥이에게 더 물어보기' 라우팅 연결 및 뒤로가기 재노출 방지 Aug 24, 2026
- FortuneDetailBottomSheet의 CTA 버튼이 TODO(#38)로 비어있어 클릭해도
  아무 동작이 없던 문제를 채팅 화면(TodakunRoute.Chat) 라우팅으로 연결
- 바텀시트가 열린 채로 채팅에 진입하면 채팅에서 뒤로가기 시 백스택에
  남아있던 detail 상태 때문에 바텀시트가 다시 열리는 버그가 있어,
  채팅 이동 전 closeDetail()로 상태를 먼저 비우도록 수정

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015do5t4ThrkmdyfYb4gkbCq
@oungsi2000
oungsi2000 force-pushed the fix/chat-cta-routing branch from be9ee57 to a5c80e8 Compare August 24, 2026 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant