[REFACTOR/#227] 유빈 2차 QA

[oilbeaneda]

Related issue 🛠

• closed [Refactor] 유빈 2차 QA #227

Work Description ✏️

• 진행 중 매칭 제안 있으면 즉시매칭 버튼이 대기화면으로 복귀 현재는 새로운 노출 조건으로 이동하는 이슈가 있었음

Screenshot 📸

Uncompleted Tasks 😅

• Task1

To Reviewers 📢

Summary by CodeRabbit

개선 사항

• 활성 강습이 없을 때 매칭 대기 중인 예약이 있으면 해당 오퍼의 대기 화면으로 이동하도록 변경했습니다.
• 매칭 대기 예약이 없으면 기존과 동일하게 새로운 매칭 화면으로 이동합니다.

문구 개선

• “강사/강습 시작” 안내 문구를 “돼요” 형태로 자연스럽게 다듬고, 관련 기본 표시값 및 프리뷰 문구도 함께 업데이트했습니다.

[coderabbitai]

Warning

Review limit reached

@oilbeaneda, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 12 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

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: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 860353b4-42c7-45ea-8342-85ae3fbdcd6f

📥 Commits

Reviewing files that changed from the base of the PR and between b2012d4 and 13878be.

📒 Files selected for processing (8)

• data/src/main/java/com/ssing/data/matching/instructormatching/model/InstructorMatchingActive.kt
• data/src/main/java/com/ssing/data/matching/instructormatching/remote/dto/response/InstructorMatchingOffersResponse.kt
• data/src/main/java/com/ssing/data/matching/instructormatching/repository/impl/InstructorMatchingRepositoryImpl.kt
• presentation/consumer-lesson/src/main/java/com/ssing/presentation/consumerlesson/mapper/LessonInfoToUiModel.kt
• presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/MatchingViewModel.kt
• presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/model/MatchingUiModel.kt
• presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/screen/MatchingPendingScreen.kt
• presentation/instructor-matching/src/main/java/com/ssing/presentation/instructormatching/screen/MatchingWaitingScreen.kt

📝 Walkthrough

Walkthrough

강사 홈의 매칭 클릭 시 매칭된 예약 오퍼가 있으면 대기 화면으로 이동하도록 변경하고, 결제·매칭 관련 표시 문구를 구어체로 수정합니다.

Changes

매칭 기능 변경

Layer / File(s)	Summary
매칭 이동 경로 선택 presentation/instructor-home/.../InstructorHomeViewModel.kt	lessonCards의 매칭된 예약 중 offerId가 있는 항목에 따라 대기 화면 또는 신규 매칭 화면으로 이동합니다.
매칭 문구 업데이트 presentation/consumer-payment/..., presentation/instructor-matching/..., core/ui/..., presentation/consumer-matching/...	결제·매칭 상태 기본값과 프리뷰, 취소 모달의 문구를 “돼요” 표현으로 변경합니다.

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

Possibly related PRs

• TEAM-SSING/SSING-ANDROID#112: InstructorHomeViewModel의 매칭 진입 및 NavigateToMatching 흐름과 연결됩니다.
• TEAM-SSING/SSING-ANDROID#164: 결제·매칭 영역의 classDateTime 문구 변경과 관련됩니다.
• TEAM-SSING/SSING-ANDROID#223: hasActiveLesson 및 NavigateToMatchingWaiting의 offerId 처리와 직접 연결됩니다.

Suggested labels: 🔨 FIX

Suggested reviewers: doyeon0307, apffkxhsls, doorimng

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (1 warning, 2 inconclusive)

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.
Title check	❓ Inconclusive	제목이 이슈 번호와 QA 맥락은 담고 있지만 변경 핵심을 충분히 설명하지 못합니다.	주요 변경사항을 드러내는 짧고 구체적인 제목으로 바꿔 주세요.
Linked Issues check	❓ Inconclusive	링크된 이슈 #227의 요구사항이 매우 모호해 코드 변경이 이슈를 충족하는지 검증하기 어렵습니다.	#227에 구체적인 완료 조건과 구현 범위를 추가해 주세요.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description check	✅ Passed	템플릿의 주요 섹션과 관련 이슈, 작업 내용, 스크린샷, 미완료 항목이 포함되어 있습니다.
Out of Scope Changes check	✅ Passed	변경은 매칭 이동 로직과 관련 문구/프리뷰 문자열 수정으로 보이며 별도 무관한 작업은 보이지 않습니다.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch refactor/#227-yubin_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.}