Skip to content

fix: 반려된 견적 요청 알림 클릭 시 견적 요청 상세 페이지로 연결 - #126

Merged
yooseohyeon merged 1 commit into
devfrom
fix/rejected-estimate-notification-link
Aug 11, 2026
Merged

yooseohyeon merged 1 commit into
devfrom
fix/rejected-estimate-notification-link

Conversation

@yooseohyeon

@yooseohyeon yooseohyeon commented Aug 11, 2026

Copy link
Copy Markdown
Member

📌 작업 내용

기사의 견적 요청 반려 알림 클릭 시, 대기 중인 견적 목록이 아닌 해당 견적 요청 상세 페이지로 이동하도록 수정했습니다.

✅ 변경 사항

  • ESTIMATE_REQUEST_REJECTED 알림의 이동 경로 수정
  • /estimates/pending/estimates/requests/${estimateRequestId}로 변경
  • 반려 알림에서 해당 견적 요청의 상세 정보 및 반려 사유를 바로 확인할 수 있도록 개선

🧪 테스트

  • 서버 실행 확인
  • API 동작 확인
  • DB 연동 확인
  • 로그 확인
  • 기타: 알림 클릭 시 상세 페이지 이동 확인

테스트 방법

  1. 기사 계정으로 견적 요청을 반려합니다.
  2. 일반 유저 계정에서 생성된 견적 요청 반려 알림을 확인합니다.
  3. 알림 클릭 시 /estimates/requests/{견적요청ID} 페이지로 이동하는지 확인합니다.

🔥 체크리스트

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

Summary by CodeRabbit

  • 개선 사항
    • 반려 알림을 선택하면 대기 중인 견적 목록이 아닌 해당 견적 요청의 상세 페이지로 바로 이동합니다.

@yooseohyeon yooseohyeon self-assigned this Aug 11, 2026
@yooseohyeon
yooseohyeon merged commit b5b6061 into dev Aug 11, 2026
@coderabbitai

coderabbitai Bot commented Aug 11, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b172b6e5-5821-459d-8d7e-a75e96595cb3

📥 Commits

Reviewing files that changed from the base of the PR and between 1fbceac and 4968e6a.

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

📝 Walkthrough

Walkthrough

견적 요청 반려 알림의 linkUrl/estimates/pending에서 해당 견적 요청 상세 페이지인 /estimates/requests/${estimateRequestId}로 변경했습니다.

Changes

견적 요청 반려 알림

Layer / File(s) Summary
반려 알림 링크 경로 변경
src/modules/estimate/mover/mover-estimate.service.ts
반려 알림의 linkUrl이 대기 중인 견적 목록에서 해당 견적 요청 상세 페이지로 변경되었습니다.

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

Possibly related PRs

Suggested labels: 🐛 fix

Suggested reviewers: soooob43

✨ 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/rejected-estimate-notification-link

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

@yooseohyeon yooseohyeon changed the title fix: 반려된 견적 요청 알림을 요청 상세로 연결 fix: 반려된 견적 요청 알림 클릭 시 견적 요청 상세 페이지로 연결 Aug 11, 2026
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