Skip to content

fix: 토너먼트 시작 후 CTA 상단 안내 토스트 제거 - #553

Merged
ychany merged 2 commits into
devfrom
fix/552-remove-waiting-toast
Aug 23, 2026
Merged

fix: 토너먼트 시작 후 CTA 상단 안내 토스트 제거#553
ychany merged 2 commits into
devfrom
fix/552-remove-waiting-toast

Conversation

@ychany

@ychany ychany commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

작업 내용

주최자가 토너먼트를 시작한 뒤에도 남아 있던 주최자가 시작해야 플레이 할 수 있어요 안내 토스트를 삭제했습니다.

주최자가 시작하면 주최자가 토너먼트를 시작했어요! 모달이 뜨므로 안내가 중복됩니다.

  • CTA 상단 안내 토스트 UI 제거
  • 토스트 노출용 state·타이머 제거
  • 버튼 비활성화·클릭 차단 동작(isWaitingForOwnerStart)은 유지

스크린샷

image

연관 이슈

closes #552

Summary by CodeRabbit

  • 변경 사항
    • 주최자 시작 전 참가자 제한 상태에 대한 설명을 명확히 했습니다.
    • 토너먼트 시작 버튼에서 참가자 안내 툴팁을 제거했습니다.
    • 토너먼트 시작, 부전승 경고, 확인 모달 및 버튼 동작은 기존과 동일하게 유지됩니다.

@vercel

vercel Bot commented Aug 23, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
piki Ready Ready Preview Aug 23, 2026 1:30pm

@github-actions github-actions Bot added fix Something isn't working WEB labels Aug 23, 2026
@github-actions

Copy link
Copy Markdown

Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다.

@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 08f7578f-a827-4541-a72a-f9a00456108e

📥 Commits

Reviewing files that changed from the base of the PR and between ecad683 and 8c3b095.

📒 Files selected for processing (2)
  • apps/web/src/app/tournament/[id]/_common/_types/tournamentResponse.ts
  • apps/web/src/app/tournament/[id]/create/_components/tournament-start-button/TournamentStartButton.tsx

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

토너먼트 시작 대기 툴팁 UI와 표시 상태 및 타이머를 제거했습니다. ownerStarted=false 상태 주석을 갱신했습니다. 토너먼트 시작, 부전승 경고, 확인 모달, 버튼 동작은 유지됩니다.

Changes

토너먼트 시작 대기 안내

Layer / File(s) Summary
시작 대기 툴팁 제거
apps/web/src/app/tournament/[id]/_common/_types/tournamentResponse.ts, apps/web/src/app/tournament/[id]/create/_components/tournament-start-button/TournamentStartButton.tsx
ownerStarted=false 상태 주석을 갱신했습니다. 툴팁 표시 상태, 지속시간 상수, useEffect 타이머 로직, 조건부 툴팁 렌더링을 제거했습니다.

Estimated code review effort: 1 (Trivial) | ~5분

Merge Risk: ⚪ Minimal · up to 8c3b0

This change removes a redundant waiting toast while preserving the button-disabled and click-blocking behavior; no actionable merge-blocking risk remains after normal checks and review.

Suggested reviewers: kanghaeun, iodio89

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목이 CTA 상단 안내 토스트 제거라는 PR의 핵심 변경을 명확히 설명합니다.
Linked Issues check ✅ Passed 토스트 UI와 노출용 state·타이머를 제거하고 isWaitingForOwnerStart 동작을 유지해 요구사항[#552]을 충족합니다.
Out of Scope Changes check ✅ Passed 변경은 토스트 관련 UI와 주석 정리에 한정되며, 링크된 이슈의 범위를 벗어난 변경이 없습니다.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/552-remove-waiting-toast

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.

@ychany
ychany merged commit a8d9087 into dev Aug 23, 2026
7 checks passed
@ychany
ychany deleted the fix/552-remove-waiting-toast branch August 23, 2026 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Something isn't working WEB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: 토너먼트 시작 후 CTA 상단 안내 토스트 제거

1 participant