Skip to content

fix: 지정견적 문구 수정 - #100

Merged
Obebe-creator merged 1 commit into
devfrom
fix/designated-request-notification-content
Aug 6, 2026
Merged

Obebe-creator merged 1 commit into
devfrom
fix/designated-request-notification-content

Conversation

@Obebe-creator

@Obebe-creator Obebe-creator commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

📌 작업 내용

지정견적 문구 수정

✅ 변경 사항

+ content: MOVE_TYPE_LABEL[request.moveType],

🧪 테스트

  • 서버 실행 확인
  • API 동작 확인
  • DB 연동 확인
  • 로그 확인
  • 기타:

테스트 방법

📷 스크린샷 (선택)

🔥 체크리스트

  • 코드 컨벤션을 준수했습니다.
  • 불필요한 console.log를 제거했습니다.
  • ESLint 오류가 없습니다.
  • 변경 사항을 직접 테스트했습니다.
  • 관련 문서를 업데이트했습니다. (필요 시)

🙏 To Reviewer

Summary by CodeRabbit

  • 개선 사항
    • 지정 견적 알림 내용이 고정 문구 대신 이사 유형별 라벨(소형이사, 가정이사, 사무실이사)로 표시됩니다.

@coderabbitai

coderabbitai Bot commented Aug 6, 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: e08ba78c-fbb5-4c5e-9a87-2fcd4e77f5ba

📥 Commits

Reviewing files that changed from the base of the PR and between f8d80b2 and a296048.

📒 Files selected for processing (1)
  • src/modules/estimate-request/estimateRequest.service.ts

📝 Walkthrough

Walkthrough

지정 견적 알림의 content가 고정 문구 대신 request.moveType에 따른 MOVE_TYPE_LABEL 값으로 생성된다.

Changes

지정 견적 알림

Layer / File(s) Summary
이사 유형별 알림 문구 반영
src/modules/estimate-request/estimateRequest.service.ts
지정 견적 알림의 contentMOVE_TYPE_LABEL의 이사 유형별 라벨을 사용한다.

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

Possibly related PRs

Suggested reviewers: youngmis, karrum5692

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 지정견적 알림 문구 수정이라는 주요 변경 사항을 간결하고 명확하게 설명합니다.
Description check ✅ Passed 필수 섹션과 변경 내용은 작성되었습니다. 테스트 방법과 일부 체크리스트 항목은 미작성 또는 미완료 상태입니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ 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 fix/designated-request-notification-content

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.

@Obebe-creator
Obebe-creator merged commit a1fb394 into dev Aug 6, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant