Skip to content

chore: 기사님 상세 SSR prefetch 디버그 로그 제거 - #110

Merged
yooseohyeon merged 1 commit into
devfrom
fix/remove-mover-detail-debug-logs
Aug 11, 2026
Merged

chore: 기사님 상세 SSR prefetch 디버그 로그 제거#110
yooseohyeon merged 1 commit into
devfrom
fix/remove-mover-detail-debug-logs

Conversation

@yooseohyeon

@yooseohyeon yooseohyeon commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

📋 작업 내용

기사님 상세 SSR prefetch 원인 파악을 위해 임시로 추가했던 디버그 코드와 로그를 제거했습니다.


🔥 변경 사항

  • MoverDetailView의 SSR prefetch 오류 전달 props 및 console.error 제거
  • 기사님 상세 페이지의 임시 오류 가공 로직 제거
  • 네트워크 요청 실패 시 디버깅용 상세 오류 정보 수집 로직 제거
  • 기존 사용자 노출 오류 처리 흐름은 유지

✅ 체크리스트

  • 로컬에서 정상 동작을 확인했습니다.
  • 기존 기능에 영향을 주지 않는지 확인했습니다.
  • 불필요한 console.log를 제거했습니다.
  • lint를 통과했습니다.
  • README 또는 문서를 수정했습니다. (필요 시)
  • API 명세와 일치하는지 확인했습니다.

Summary by CodeRabbit

  • 버그 수정
    • 상세 정보의 초기 로딩에 실패하더라도 페이지가 중단되지 않고 화면을 계속 표시합니다.
    • 초기 요청이 실패한 경우 클라이언트에서 데이터를 다시 요청해 정상적으로 정보를 불러올 수 있습니다.
    • 네트워크 오류 발생 시 보다 일관된 기본 오류 메시지를 제공합니다.
    • 요청 취소 및 시간 초과 상황의 기존 처리는 유지됩니다.

@yooseohyeon yooseohyeon self-assigned this Aug 11, 2026
@vercel

vercel Bot commented Aug 11, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
moving-frontend Ready Ready Preview Aug 11, 2026 1:04am
moving-frontend-p2ol Ready Ready Preview Aug 11, 2026 1:04am

@yooseohyeon
yooseohyeon merged commit 37b456b into dev Aug 11, 2026
3 of 4 checks passed
@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: 410d4fc6-6d70-4137-87a1-bbb902b85975

📥 Commits

Reviewing files that changed from the base of the PR and between 367b0b4 and 0239626.

📒 Files selected for processing (3)
  • src/app/movers/[moverId]/page.tsx
  • src/components/mover/detail/MoverDetailView.tsx
  • src/lib/api/fetchInstance.ts

📝 Walkthrough

Walkthrough

Mover 상세 페이지는 SSR 프리패치 실패를 별도 오류 객체로 전달하지 않습니다. 프리패치 실패 후에도 페이지를 렌더링하고 클라이언트 재요청을 사용합니다. 네트워크 오류의 상세 원인 전달도 제거했습니다.

Changes

Mover 상세 데이터 프리패치

Layer / File(s) Summary
SSR 프리패치와 상세 뷰 연결
src/app/movers/[moverId]/page.tsx, src/components/mover/detail/MoverDetailView.tsx
SSR 상세 데이터 프리패치를 try/catch로 처리합니다. 실패해도 페이지 렌더링을 계속합니다. MoverDetailViewmoverIdinitialDetail만 받습니다.
네트워크 오류 생성 단순화
src/lib/api/fetchInstance.ts
네트워크 오류 원인 추출을 제거합니다. ApiError에는 기본 네트워크 오류 메시지만 전달합니다. Abort 및 timeout 처리는 유지합니다.

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

Sequence Diagram(s)

sequenceDiagram
  participant MoverDetailPage
  participant MoverDetailView
  participant fetchInstance
  MoverDetailPage->>fetchInstance: 상세 데이터 SSR 프리패치
  fetchInstance-->>MoverDetailPage: 성공 또는 오류
  MoverDetailPage->>MoverDetailView: moverId와 initialDetail 전달
  MoverDetailView->>fetchInstance: 실패 시 상세 데이터 재요청
Loading

Possibly related PRs

✨ 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/remove-mover-detail-debug-logs

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

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