Skip to content

feat: 친구 토너먼트 결과에도 영수증 출력기 GUI 적용 - #547

Merged
ychany merged 1 commit into
devfrom
feat/546-group-result-draw-machine
Aug 23, 2026
Merged

feat: 친구 토너먼트 결과에도 영수증 출력기 GUI 적용#547
ychany merged 1 commit into
devfrom
feat/546-group-result-draw-machine

Conversation

@ychany

@ychany ychany commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 친구 토너먼트 결과 화면에도 영수증 출력기 GUI·애니메이션을 적용합니다.

작업 세부 내용

개인 결과에만 있던 출력기가 친구 결과에는 없어 같은 영수증인데 화면마다 표현이 달랐습니다.

ReceiptDrawMachine 일반화

기존에는 ReceiptPaper 를 직접 렌더하고 있어 재사용할 수 없었습니다. 친구 결과는 참여자 칩·펼치기 등 내용 구조가 달라 같은 props 로는 그릴 수 없기 때문입니다.

영수증 본문을 children 으로 받도록 바꿨습니다.

// 전 — 개인 결과 전용
<ReceiptDrawMachine tournamentId={...} result={...} date={...} />

// 후 — 내용을 주입
<ReceiptDrawMachine>
  <ReceiptPaper ... />           {/* 개인 결과 */}
</ReceiptDrawMachine>

<ReceiptDrawMachine>
  <div>...친구 결과 영수증...</div>
</ReceiptDrawMachine>

프린터 프레임·슬롯 애니메이션·마스킹 로직을 두 화면이 공유합니다. 중복이 사라져 -95 / +81 로 줄었습니다.

검증

타입·린트 통과했습니다.

tournamentResult.spec.ts 는 4개 실패하지만 dev 기준선과 동일합니다(stash 후 대조 확인). 이번 변경으로 인한 회귀는 아닙니다.

확인 필요

친구 결과 영수증은 참여자 칩이 펼쳐지면 개인 결과보다 길어집니다. 긴 영수증에서 출력기 애니메이션이 자연스러운지는 실제 화면 확인이 필요합니다.

스크린샷

스크린샷 2026-08-23 오후 9 23 29

연관 이슈

closes #546

개인 결과에만 있던 출력기 GUI·애니메이션을 친구 결과에도 붙인다.

ReceiptDrawMachine 이 ReceiptPaper 를 직접 렌더하고 있어 재사용이
불가능했다. 영수증 본문을 children 으로 받도록 일반화했다.
@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 12:25pm

@github-actions github-actions Bot added the feature New feature or request label Aug 23, 2026
@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the WEB label Aug 23, 2026
@coderabbitai

coderabbitai Bot commented Aug 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@ychany, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 41 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: c674c0ee-a2d0-4a06-b8c6-0e87768fc6ce

📥 Commits

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

📒 Files selected for processing (3)
  • apps/web/src/app/tournament/[id]/result/_components/ReceiptDrawMachine.tsx
  • apps/web/src/app/tournament/[id]/result/_components/ResultClient.tsx
  • apps/web/src/app/tournament/[id]/result/group/_components/GroupResultClient.tsx

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.

@github-actions
github-actions Bot requested a review from kanghaeun August 23, 2026 12:25
@ychany
ychany merged commit 1aebba4 into dev Aug 23, 2026
9 checks passed
@ychany
ychany deleted the feat/546-group-result-draw-machine branch August 23, 2026 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request WEB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: 친구 토너먼트 결과에도 영수증 출력기 GUI 적용

1 participant