Skip to content

style: (QA/3) 매칭 생성 완료 화면에 생성자 표시 추가#371

Merged
heesunee merged 1 commit intodevelopfrom
style/#334/owner
Sep 7, 2025
Merged

style: (QA/3) 매칭 생성 완료 화면에 생성자 표시 추가#371
heesunee merged 1 commit intodevelopfrom
style/#334/owner

Conversation

@Dubabbi
Copy link
Contributor

@Dubabbi Dubabbi commented Sep 7, 2025

#️⃣ Related Issue

Closes #334

☀️ New-insight

이전에 MatchCurrentCard에 추가했던 isCreated 플래그를 실제 호출부(그룹/1:1 생성 완료 섹션)에서 활용하도록 반영했습니다.
이제 생성 완료 화면의 카드들은 isCreated에 따라 생성자 왕관을 표시합니다!!

💎 PR Point

group-match-card-section, single-match-card-section의 카드 호출부에 isCreated를 명시적으로 추가하였습니다.

📸 Screenshot

화면 캡처 2025-09-07 205016 화면 캡처 2025-09-07 205035

Summary by CodeRabbit

  • New Features
    • 생성된 매치 카드에 ‘생성됨’ 상태가 표시되어 방금 만든 매치를 한눈에 식별할 수 있습니다.
    • 단일/그룹 매치 생성 화면 모두 동일한 표시가 적용되어 흐름이 더 일관적입니다.
  • Style
    • 카드의 상태 표시가 시각적으로 개선되어 가독성과 구분감이 향상되었습니다.

@Dubabbi Dubabbi self-assigned this Sep 7, 2025
@coderabbitai
Copy link

coderabbitai bot commented Sep 7, 2025

Walkthrough

Card 컴포넌트에 선택적 불리언 prop isCreated가 추가되었고, 매칭 생성 화면의 단일/그룹 카드 섹션에서 Card를 렌더링할 때 isCreated를 전달하도록 수정되었습니다. 해당 타입 선언(CardProps)도 업데이트되었습니다.

Changes

Cohort / File(s) Change Summary
Match creation pages
src/pages/match/create/components/single-match-card-section.tsx, src/pages/match/create/components/group-match-card-section.tsx
두 섹션에서 <Card ... /> 호출 시 isCreated prop 추가 전달
Card component API
src/components/card/match-card/card.tsx, src/components/card/match-card/types/card.ts
CardPropsisCreated?: boolean 추가, CardisCreated prop을 수용하도록 공개 API 확장

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Assessment against linked issues

Objective Addressed Explanation
매칭 생성 화면에 생성자 표시를 추가 (#334)

Assessment against linked issues: Out-of-scope changes

(해당 없음)

Possibly related PRs

Suggested labels

style, 소은

Suggested reviewers

  • yeeeww
  • bongtta

Poem

새 카드 깃발 휘날려—isCreated라구요!
폴짝폴짝 생성자 표식, 딱 보이게 보여줘요.
단짝 카드도, 그룹 카드도, 모두 체크 완료!
(ง•̀_•́)ง 토끼는 승인 버튼을 살짝—클릭!
오늘도 머지의 밭에 당근이 주렁주렁 🥕

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch style/#334/owner

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 and usage tips.

@github-actions github-actions bot added style css styling 소은 labels Sep 7, 2025
@Dubabbi Dubabbi changed the title style: (QA/3) 매칭 생성 화면에 생성자 표시 추가 style: (QA/3) 매칭 생성 완료 화면에 생성자 표시 추가 Sep 7, 2025
@github-actions
Copy link

github-actions bot commented Sep 7, 2025

MATEBALL-STORYBOOK
⚾ Storybook 배포가 완료되었습니다!

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (2)
src/pages/match/create/components/single-match-card-section.tsx (1)

40-40: className 중복 전달 제거 제안

Line 35의 cardProps.className가 이미 'w-full'로 설정되어 있어 Line 40의 className="w/full"는 스프레드에 의해 덮입니다. 혼동을 줄이기 위해 JSX의 className를 제거하는 편이 깔끔합니다.

-      <Card isCreated className="w-full" {...cardProps} />
+      <Card isCreated {...cardProps} />
src/pages/match/create/components/group-match-card-section.tsx (1)

38-38: className 이중 지정 정리

Line 33에서 cardProps.className가 이미 'w-full'이므로, Line 38의 className는 스프레드에 의해 무시됩니다. 한 곳으로 통일하세요.

-      <Card isCreated className="w-full" {...cardProps} />
+      <Card isCreated {...cardProps} />
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 84c4226 and db08d68.

📒 Files selected for processing (2)
  • src/pages/match/create/components/group-match-card-section.tsx (1 hunks)
  • src/pages/match/create/components/single-match-card-section.tsx (1 hunks)
🔇 Additional comments (2)
src/pages/match/create/components/single-match-card-section.tsx (1)

40-40: isCreated 전달 적절합니다

생성 화면에서 생성자 표시 목적에 부합하며, Card에 새 prop이 노출되었다는 전제에서 사용 일관성도 좋습니다.

src/pages/match/create/components/group-match-card-section.tsx (1)

38-38: isCreated 추가 사용 LGTM

그룹 생성 섹션에서도 생성자 표시가 일관되게 적용되었습니다.

@heesunee heesunee merged commit f4dc955 into develop Sep 7, 2025
7 checks passed
@heesunee heesunee deleted the style/#334/owner branch September 7, 2025 12:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

style css styling 소은

Projects

None yet

Development

Successfully merging this pull request may close these issues.

style: 매칭 생성 화면에 생성자 표시를 추가합니다.

2 participants