Skip to content

fix: 다회모드/블라인드 세션 완료 시 모드 OFF 처리#277

Merged
FLYLIKEB merged 1 commit intomainfrom
fix/session-mode-clear-on-complete
Mar 18, 2026
Merged

fix: 다회모드/블라인드 세션 완료 시 모드 OFF 처리#277
FLYLIKEB merged 1 commit intomainfrom
fix/session-mode-clear-on-complete

Conversation

@FLYLIKEB
Copy link
Copy Markdown
Owner

@FLYLIKEB FLYLIKEB commented Mar 18, 2026

Summary

  • 세션 완료 후에도 ModeCarousel에서 ON 상태가 유지되는 버그 수정
  • SessionSummary: 노트 발행 성공 시 clearSession() 호출
  • BlindSessionDetail: 세션 종료 시 clearBlind() 호출
  • SessionInProgress/BlindSessionDetail: 이미 완료된 세션 방문 시에도 clear 호출

Test plan

  • npm run build
  • 로컬 실행 확인

🤖 Generated with Claude Code

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 블라인드 세션 종료 시 앱 상태가 자동으로 초기화되어 상태 충돌 방지
    • 노트 발행 시 이전 세션 상태가 자동으로 정리됨
    • 세션 발행 완료 후 상태 정리 개선으로 전반적인 안정성 향상

세션 완료 후에도 ModeCarousel에서 ON 상태가 유지되는 버그 수정
- SessionSummary: 노트 발행 성공 시 clearSession() 호출
- BlindSessionDetail: 세션 종료 시 clearBlind() 호출
- SessionInProgress/BlindSessionDetail: 이미 완료된 세션 방문 시에도 clear 호출

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cha-log Building Building Preview Mar 18, 2026 3:31am

@FLYLIKEB FLYLIKEB merged commit f129ede into main Mar 18, 2026
2 checks passed
@FLYLIKEB FLYLIKEB deleted the fix/session-mode-clear-on-complete branch March 18, 2026 03:31
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 18, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c15b1048-0854-48cc-953f-ef4cabbb2d02

📥 Commits

Reviewing files that changed from the base of the PR and between a94d66d and 22b6ec5.

📒 Files selected for processing (3)
  • src/pages/BlindSessionDetail.tsx
  • src/pages/SessionInProgress.tsx
  • src/pages/SessionSummary.tsx

Walkthrough

세 개의 페이지 컴포넌트에서 AppModeContext의 useAppMode 훅을 사용하여 세션 및 블라인드 모드 상태를 정리하는 기능을 추가했습니다. 세션 종료 후, 세션 발행 후, 또는 노트 발행 후 자동으로 관련 상태를 초기화합니다.

Changes

Cohort / File(s) Summary
AppMode 상태 정리 기능 추가
src/pages/BlindSessionDetail.tsx, src/pages/SessionInProgress.tsx, src/pages/SessionSummary.tsx
useAppMode 훅을 임포트하고 clearBlind/clearSession 함수를 추출하여 세션 종료, 노트 발행, 또는 noteId 변경 시점에 상태 정리를 자동으로 수행하도록 수정.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/session-mode-clear-on-complete
📝 Coding Plan
  • Generate coding plan for human review comments

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 and usage tips.

Tip

CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.

Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present.

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.

1 participant