Skip to content

feat: 결과 하단 CTA 라벨·아이콘을 시안 기준으로 변경 (영수증 공유·트로피) - #414

Merged
ychany merged 2 commits into
devfrom
feat/400-result-cta
Aug 1, 2026
Merged

feat: 결과 하단 CTA 라벨·아이콘을 시안 기준으로 변경 (영수증 공유·트로피)#414
ychany merged 2 commits into
devfrom
feat/400-result-cta

Conversation

@ychany

@ychany ychany commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 토너먼트 결과 하단 CTA 를 시안 기준으로 변경합니다 (영수증 공유 라벨·아이콘)

작업 세부 내용

  • 결과 화면 하단 CTA 의 라벨과 아이콘을 변경했습니다.
버튼 변경 전 변경 후
왼쪽 "영수증 저장" + 다운로드 아이콘 "영수증 공유" + 영수증 아이콘 (ReciptIconFill)
오른쪽 "토너먼트 공유" + 업로드 아이콘 라벨 동일 + 트로피 아이콘 (TrophyIconFill)
  • 동작은 기존 그대로 유지합니다. 왼쪽 버튼을 누르면 기존처럼 영수증을 캡처해 네이티브 공유 시트(미지원 환경은 다운로드)를 띄웁니다.
  • 시안의 영수증 공유 바텀시트(이미지 저장 · 이미지 복사 · 채팅방에 공유 · 스토리 공유)는 feat: 위시/토너먼트 아이템 수기 수정 상시 허용 대응 #416 에서 구현하며, 이 CTA 가 해당 바텀시트를 여는 트리거로 연결됩니다.
  • 라벨 변경에 맞춰 e2e 스펙의 버튼 단언도 함께 수정했습니다 (영수증 저장 → 영수증 공유).

연관 이슈

closes #400

Summary by CodeRabbit

  • 새 기능
    • 결과 화면의 영수증 CTA가 ‘영수증 저장’에서 ‘영수증 공유’로 변경되었습니다.
    • 토너먼트 관련 아이콘이 트로피 아이콘으로 업데이트되어 의미를 더 명확하게 전달합니다.

@vercel

vercel Bot commented Aug 1, 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 1, 2026 2:41pm

@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown

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

@github-actions
github-actions Bot requested a review from kanghaeun August 1, 2026 14:38
@github-actions github-actions Bot added feature New feature or request WEB labels Aug 1, 2026
@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

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

Next review available in: 58 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 80bd5cf2-b9f0-4747-9f92-d9841b484217

📥 Commits

Reviewing files that changed from the base of the PR and between 2def7cd and 200b604.

📒 Files selected for processing (1)
  • apps/web/src/app/tournament/[id]/result/_components/ResultClient.tsx
📝 Walkthrough

Walkthrough

토너먼트 결과 화면의 영수증 CTA가 저장에서 공유로 변경되었습니다. 아이콘과 관련 E2E 테스트도 새 표시 기준에 맞게 갱신되었습니다. 토너먼트 공유 CTA 아이콘은 트로피 아이콘으로 변경되었습니다.

Changes

토너먼트 결과 CTA

Layer / File(s) Summary
결과 CTA 표시 변경
apps/web/src/app/tournament/[id]/result/_components/ResultClient.tsx
영수증 CTA의 문구와 아이콘을 영수증 공유 및 영수증 아이콘으로 변경했습니다. 토너먼트 공유 CTA 아이콘을 트로피 아이콘으로 변경했습니다.
CTA E2E 검증 갱신
apps/web/e2e/specs/tournament/tournamentResult.spec.ts
테스트 설명과 선택자를 영수증 저장 버튼에서 영수증 공유 버튼 기준으로 변경했습니다.

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

Possibly related issues

  • TeamPiKi/client 399번 이슈: 토너먼트 결과 화면의 영수증 공유 UI와 CTA 문구 변경을 다룹니다.

Possibly related PRs

  • TeamPiKi/client#358: 같은 ResultClient.tsx 영수증 CTA의 저장-공유 변경과 관련됩니다.
  • TeamPiKi/client#389: 같은 tournamentResult.spec.ts의 영수증 CTA E2E 검증 변경과 관련됩니다.

Suggested labels: style, test

Suggested reviewers: kanghaeun

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning [#400] 라벨과 아이콘은 변경했지만, 왼쪽 CTA를 영수증 공유 바텀시트 오픈 트리거로 변경한 코드가 없습니다. 왼쪽 CTA의 클릭 동작을 영수증 공유 바텀시트 오픈으로 연결하세요.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 결과 화면 하단 CTA의 라벨과 아이콘 변경이라는 주요 변경 사항을 명확하게 설명합니다.
Out of Scope Changes check ✅ Passed 변경된 E2E 테스트와 결과 화면 CTA 수정은 연결 이슈의 라벨 및 아이콘 변경 범위에 포함됩니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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 feat/400-result-cta

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 3ed7c59 into dev Aug 1, 2026
7 checks passed
@ychany
ychany deleted the feat/400-result-cta branch August 1, 2026 17:22
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: 토너먼트 결과 하단 CTA를 "영수증 저장"에서 "영수증 공유"(바텀시트 오픈)로 변경

1 participant