fix: 토너먼트 준비 화면 참여자 패널 디자인 조정 및 '나' 표시 - #538
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Review limit reached
Next review available in: 13 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 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 configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
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. Comment |
|
Discord 스레드 연동용 메타데이터입니다. discord-pr-bot 워크플로가 자동 생성하며, 수정·삭제하면 PR 과 Discord 알림 연동이 끊깁니다. |
작업 요약
작업 세부 내용
1. 담기 카운트다운 칩 디자인 보정 —
DepositCountdown.tsxtext-text-accent→text-text-accent/88, 타이머 아이콘은text-sky-blue-400으로 분리해 시안과 맞췄습니다gap-1.5→gap-1,pl-2→pl-2.5timer.svg에 흰색 원 배경을 추가해 아이콘 몸통이 시안처럼 채워져 보이게 했습니다showLabel=true)에도 동일하게 적용되는 변경이며, 두 사용처 모두 시안과 대조해 확인했습니다2. 참여자 칩 본인 표기 —
ParticipantChip.tsxuseGetMe()의userData.id와 칩의user.id(참여자userId)를 비교해 본인이면 "나"를 노출합니다3.
UserT.name필수화 (부수 정리)nickname: string이 항상 내려오는데, 표시용UserT.name이 optional이라 도달 불가능한?? '익명'fallback이 있었습니다UserT를 만드는 곳(ParticipantPanel매핑,FriendListDialog의toUser) 모두name을 항상 채우고 있어name: string필수로 좁히고 fallback을 제거했습니다UserProfile의alt도`${user.name} 프로필 이미지`로 정리했습니다스크린샷
연관 이슈
closes #536