[REFACTOR/#224] 도연 2차 QA

[doyeon0307]

Related issue 🛠

• closed [Refactor] 도연 2차 QA #224

Work Description ✏️

• 분홍색 친 거 제외 여기서 반영했어요

Screenshot 📸

Uncompleted Tasks 😅

• Task1

To Reviewers 📢

Summary by CodeRabbit

• UI 개선

  • 소비자 정보 카드의 항목 간 간격을 조정해 가독성을 높였습니다.
  • 수업 예약 정보의 날짜와 위치 표시를 간결하게 정리했습니다.
  • 매칭 대기 화면의 상세 카드 테두리 스타일을 개선했습니다.

• 매칭 개선

  • 매칭 참가 팀이 준비 완료 전 상태로 정확히 표시됩니다.
  • 리뷰 화면에 키워드 정보가 표시됩니다.

[coderabbitai]

📝 Walkthrough

Walkthrough

소비자·강사 매칭 관련 카드의 레이아웃과 스타일이 조정되었습니다. 참가자 준비 상태 매핑이 변경되었고, 리뷰 상태에 키워드가 추가되었으며, 예약 날짜 영역의 구분선이 제거되었습니다.

Changes

매칭 및 카드 UI

Layer / File(s)	Summary
레슨 카드 레이아웃 core/ui/src/main/java/com/ssing/core/ui/common/component/ConsumerInfoCard.kt, core/ui/src/main/java/com/ssing/core/ui/common/component/HomeLessonCardList.kt	소비자 정보 카드의 세로 간격이 4.dp에서 6.dp로 변경되고, 예약 날짜 영역의 구분선 아이콘이 제거되었습니다.
소비자 매칭 상태 presentation/consumer-lesson/src/main/java/com/ssing/presentation/consumerlesson/mapper/LessonMatchingRequestToUiModel.kt, presentation/consumer-matching/src/main/java/com/ssing/presentation/consumermatching/ConsumerMatchingViewModel.kt	참가자 팀의 isReady가 false로 매핑되며, 리뷰 상태에 두 개의 키워드가 설정됩니다.
강사 매칭 카드 표시 presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/screen/MatchingWaitingScreen.kt, presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/screen/MatchingOfferScreen.kt	대기 화면 카드에 대체 테두리 색상이 지정되고, 제안 화면의 공백이 조정되었습니다.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• TEAM-SSING/SSING-ANDROID#69: HomeLessonCardList.kt의 예약·날짜 영역 UI 변경과 직접 연결됩니다.
• TEAM-SSING/SSING-ANDROID#76: ConsumerInfoCard.kt의 레이아웃 변경과 직접 연결됩니다.
• TEAM-SSING/SSING-ANDROID#152: ParticipantTeamUiModel.isReady 매핑 변경과 직접 연결됩니다.

Suggested labels: ♻️ REFACTOR, 🧱Component

Suggested reviewers: oilbeaneda, joyrii

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	직접 연결된 #224의 2차 QA 범위에 맞는 UI/상태 수정들이 반영되어 있어 이슈 목표를 충족합니다.
Out of Scope Changes check	✅ Passed	요약된 변경들은 모두 2차 QA에서 다룰 수 있는 UI 정리와 상태 보정 범위 안에 있습니다.
Title check	✅ Passed	제목은 #224 기반의 2차 QA 반영이라는 주요 변경과 일치해 간결하고 관련성이 높습니다.
Description check	✅ Passed	필수 섹션(관련 이슈, 작업 내용, 스크린샷, 미완료 작업, 리뷰어)이 모두 포함되어 템플릿을 대부분 충족합니다.

✨ 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 refactor/#224-dodo_qa_2

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}