Skip to content

[MOD/#214] 소비자 매칭 결과 평점 목데이터 고정 - #217

Merged
doyeon0307 merged 1 commit into
developfrom
mod/#214-consumer_rating_mock
Jul 16, 2026
Merged

doyeon0307 merged 1 commit into
developfrom
mod/#214-consumer_rating_mock

Conversation

@doyeon0307

@doyeon0307 doyeon0307 commented Jul 16, 2026 •

Copy link
Copy Markdown
Collaborator

Related issue 🛠

Work Description ✏️

  • 소비자 매칭 결과 평점 목데이터 고정

Screenshot 📸

image

Uncompleted Tasks 😅

  • Task1

To Reviewers 📢

흐아

Summary by CodeRabbit

  • 개선 사항
    • 소비자 매칭 화면의 초기 상태에 기본 평점 4.8이 표시됩니다.
    • 활성 상태를 불러올 때 기존 강사 평균 평점으로 평점이 자동 갱신되지 않도록 변경되었습니다.

@doyeon0307 doyeon0307 linked an issue Jul 16, 2026 that may be closed by this pull request
1 task done
@doyeon0307
doyeon0307 requested a review from a team July 16, 2026 09:05
@doyeon0307 doyeon0307 self-assigned this Jul 16, 2026
@coderabbitai

coderabbitai Bot commented Jul 16, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1116e8fd-cc04-4159-adc5-858b57a76dcd

📥 Commits

Reviewing files that changed from the base of the PR and between 2c380a8 and ca38245.

📒 Files selected for processing (1)
  • presentation/consumer-matching/src/main/java/com/ssing/presentation/consumermatching/ConsumerMatchingViewModel.kt

📝 Walkthrough

Walkthrough

소비자 매칭 결과의 초기 평점을 "4.8"로 설정하고, 활성 상태 병합 시 instructor 프로필 평균 평점으로 덮어쓰던 동작을 제거했습니다.

Changes

소비자 매칭 평점

Layer / File(s) Summary
평점 상태 설정 및 병합
presentation/consumer-matching/.../ConsumerMatchingViewModel.kt
loadReview() 초기 상태에 "4.8"을 설정하고, State.mergeFrom(active)에서 평점 갱신 로직을 제거했습니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 소비자 매칭 결과의 평점 목데이터를 4.8로 고정하는 변경을 구체적으로 잘 요약합니다.
Description check ✅ Passed 관련 이슈, 작업 내용, 스크린샷, 미완료 작업, 리뷰어 메모가 템플릿 구조를 대부분 충족합니다.
Linked Issues check ✅ Passed 직접 연결된 #214의 요구사항인 평점 더미 4.8 고정이 코드 변경으로 반영되었습니다.
Out of Scope Changes check ✅ Passed 요구사항과 무관한 변경은 보이지 않으며, 평점 목데이터 고정 범위에 맞는 수정만 포함됩니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ 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 mod/#214-consumer_rating_mock

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.

@oilbeaneda oilbeaneda left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Image

@joyrii joyrii left a comment •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Image

@doyeon0307
doyeon0307 merged commit 592b7aa into develop Jul 16, 2026
2 checks passed
@doyeon0307
doyeon0307 deleted the mod/#214-consumer_rating_mock branch July 16, 2026 09:17
@apffkxhsls apffkxhsls added this to the [Ssing] APPJAM milestone Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Mod] 소비자 매칭 결과 평점 목데이터 넣기

4 participants