Skip to content

fix: 친구 목록 모달의 딤 클릭·조회 실패 처리 - #507

Merged
ychany merged 2 commits into
devfrom
fix/506-friend-list-dialog
Aug 12, 2026
Merged

fix: 친구 목록 모달의 딤 클릭·조회 실패 처리#507
ychany merged 2 commits into
devfrom
fix/506-friend-list-dialog

Conversation

@ychany

@ychany ychany commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

작업 요약

  • 친구 목록 모달이 딤 클릭으로 닫히도록 수정합니다.
  • 조회 실패를 "친구 없음" 과 구분해 안내합니다.

작업 세부 내용

1. 딤 클릭이 차단되던 문제

MorePopover 의 루트 div 가 하위 클릭을 전부 막고 있었습니다. 카드 전체가 <Link> 라 더보기 조작이 페이지 이동으로 이어지지 않게 하려는 처리인데, Dialog 도 이 안에 있어 딤 클릭 이벤트가 함께 죽었습니다.

Dialog 를 래퍼 밖으로 옮겼습니다. 카드 이동 차단은 Popover 영역에만 적용됩니다.

<>
  <div className="contents" onClick={handleStopCardNavigation}>
    <Popover>...</Popover>
  </div>

  <TournamentDeleteDialog ... />
  <FriendListDialog ... />
</>

TournamentDeleteDialog 도 같은 문제였어서 함께 옮겼습니다.

2. 조회 실패가 "친구 없음" 으로 표시되던 문제

useGetGroupResultisGroupResultError 를 반환하는데 모달이 구독하지 않았습니다. 에러 시 groupResultDataundefined → 빈 배열 → "참여한 친구가 없어요" 로 떨어졌습니다.

실패 상태를 별도 문구로 분리했습니다.

실패 → "친구 목록을 불러오지 못했어요."
빈값 → "참여한 친구가 없어요."

문구는 의도적으로 일반적인 표현을 썼습니다. 실패 사유가 401·403·404·409·네트워크로 여러 갈래라 특정 사유로 단정할 수 없고, 특히 409 는 서버 응답과 실제 사유가 어긋나 있어(#504 참고) 그 위에 구체적 안내를 얹기 어렵습니다.

연관 이슈

closes #506

- MorePopover 의 stopPropagation 래퍼 안에 Dialog 가 있어 딤 클릭이
  막혔다. Dialog 를 래퍼 밖으로 빼 카드 이동 차단은 유지하면서
  모달은 정상적으로 닫히게 한다.
- useGetGroupResult 의 isGroupResultError 를 쓰지 않아 조회 실패가
  "참여한 친구가 없어요" 로 표시됐다. 실패 안내를 분리한다.
@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:22pm

@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: 44 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: d7fe27d0-191a-4268-a814-5e5782b7510a

📥 Commits

Reviewing files that changed from the base of the PR and between 01b317c and 29d8872.

📒 Files selected for processing (2)
  • apps/web/src/components/tournament-card/FriendListDialog.tsx
  • apps/web/src/components/tournament-card/MorePopover.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 added fix Something isn't working WEB labels Aug 12, 2026
@github-actions
github-actions Bot requested a review from iOdiO89 August 12, 2026 16:36
@github-actions

Copy link
Copy Markdown

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

@ychany
ychany merged commit d3199ac into dev Aug 12, 2026
6 of 7 checks passed
@ychany
ychany deleted the fix/506-friend-list-dialog branch August 12, 2026 17:18
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