Skip to content

fix: 참여자·게스트도 전체 결과 보기를 볼 수 있도록 수정 - #505

Merged
ychany merged 4 commits into
devfrom
fix/504-group-result-for-participants
Aug 12, 2026
Merged

fix: 참여자·게스트도 전체 결과 보기를 볼 수 있도록 수정#505
ychany merged 4 commits into
devfrom
fix/504-group-result-for-participants

Conversation

@ychany

@ychany ychany commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 전체 결과 보기 카드를 참여자·플레이 링크 게스트에게도 노출합니다.
  • CLONE 에서 진입할 때 원본 토너먼트 id 로 그룹 결과를 조회합니다.
  • 혼자 끝낸 토너먼트에서는 카드를 숨깁니다.

작업 세부 내용

1. 참여자·게스트에게 노출

카드 노출 조건이 isRoot 를 요구해 주최자에게만 보였습니다.

참여자와 게스트는 담기 화면에서 "시작" 을 누르는 순간 본인 CLONE 이 생성되고 그 인스턴스로 진행합니다. 결과 화면에 도달할 때는 CLONE 이라 isRoot=false 가 되어 항상 걸러졌습니다.

수정 전

주최자   ROOT 에서 진행 → 결과도 ROOT   → isRoot=true   ✅ 노출
참여자   CLONE 생성 → 결과는 CLONE      → isRoot=false  ❌ 숨김
게스트   CLONE 생성 → 결과는 CLONE      → isRoot=false  ❌ 숨김

수정 후 — 조건에서 isRoot 를 제거해 전원 노출

주최자   isRoot=true    ✅ 노출
참여자   isRoot=false   ✅ 노출
게스트   isRoot=false   ✅ 노출

2. 조회 대상 id 보정

그룹 결과는 원본(ROOT) 단위로 집계되므로 CLONE 에서는 원본 id 로 조회해야 합니다.

const groupResultTournamentId = tournamentData.sourceTournamentId ?? tournamentId;

dev 서버로 실측 확인했습니다. CLONE id 로 호출하면 403 이 납니다.

id isRoot source group-result
88 false 84 403
49 false 46 403

보정 없이는 참여자가 카드를 눌러도 403 만 나옵니다.

3. 혼자 끝낸 토너먼트는 숨김

hasGroupResult 검사가 게스트에게만 걸려 있어, 회원은 완료한 CLONE 이 없어도 카드가 보였습니다. 누르면 서버가 409 를 줍니다.

실측 결과 혼자 끝낸 토너먼트 11건 모두 hasGroupResult=false + group-result 409 였고, CLONE 이 있는 2건만 true 였습니다. 회원·게스트 구분 없이 이 값으로 막도록 바꿨습니다.

연관 이슈

closes #504

ychany added 3 commits August 13, 2026 00:48
노출 조건이 isRoot 를 요구해 주최자에게만 카드가 보였다.
참여자와 플레이 링크 게스트는 시작 시 본인 CLONE 이 생겨
isRoot=false 가 되므로 항상 걸러졌다.

- 노출 조건에서 isRoot 제거
- 그룹 결과는 원본(ROOT) 단위라 CLONE 이면 sourceTournamentId 로 조회
hasGroupResult 검사를 게스트에게만 적용해, 회원은 완료한 CLONE 이
없어도 카드가 보였다. 누르면 서버가 409(TOURNAMENT-028) 를 준다.

dev 서버 실측으로 확인 — 혼자 끝낸 토너먼트 11건 모두
hasGroupResult=false + group-result 409, CLONE 이 있는 2건만 true.
@vercel

vercel Bot commented Aug 12, 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 12, 2026 5:19pm

@github-actions

Copy link
Copy Markdown

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

@github-actions
github-actions Bot requested a review from soyeong0115 August 12, 2026 16:01
@ychany ychany changed the title Fix/504 group result for participants fix: 참여자·게스트도 전체 결과 보기를 볼 수 있도록 수정 Aug 12, 2026
@coderabbitai

coderabbitai Bot commented Aug 12, 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: 46 minutes

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?

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: 73bdd9e0-ad7a-4956-a809-5055ec2fff8a

📥 Commits

Reviewing files that changed from the base of the PR and between 199a640 and 2541bd2.

📒 Files selected for processing (1)
  • apps/web/src/app/tournament/[id]/result/_components/ResultClient.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.

@ychany
ychany merged commit 01b317c into dev Aug 12, 2026
6 of 7 checks passed
@ychany
ychany deleted the fix/504-group-result-for-participants branch August 12, 2026 17:16
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: 참여자 결과 화면에 전체 결과 보기가 노출되지 않음

1 participant