Skip to content

[Refactor/#434] 회원 탈퇴 에러 메세지 매핑 제거 - #435

Merged
jjjsun merged 1 commit into
developfrom
refactor/#434
Aug 10, 2026
Merged

[Refactor/#434] 회원 탈퇴 에러 메세지 매핑 제거#435
jjjsun merged 1 commit into
developfrom
refactor/#434

Conversation

@jjjsun

@jjjsun jjjsun commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

🚨 관련 이슈

Closed #434

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

  • useDeleteMyAccount.tsWITHDRAW_ERROR_MESSAGE 제거
  • IApiErrorResponse.message를 바로 사용하도록 변경

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

Swagger 에러 메세지가 문장으로 끝나지 않아서 UX용 code 매핑 넣었었는데,
프로젝트 공통 에러 처리 방식이랑 맞지 않아서 apiError.message 사용으로 통일했습니다.
추후에 필요하면 백엔드 메세지 문구를 다듬는 방향으로 논의 해보면 좋을것같습니다!

💬 리뷰어 가이드 (P-Rules)
P1: 필수 반영 (Critical) - 버그 가능성, 컨벤션 위반. 해결 전 머지 불가.
P2: 적극 권장 (Recommended) - 더 나은 대안 제시. 가급적 반영 권장.
P3: 제안 (Suggestion) - 아이디어 공유. 반영 여부는 드라이버 자율.
P4: 단순 확인/칭찬 (Nit) - 사소한 오타, 칭찬 등 피드백.

Summary by CodeRabbit

  • 버그 수정
    • 회원 탈퇴 실패 시 API가 반환한 오류 메시지를 우선 표시합니다.
    • 응답 메시지가 없는 경우 기존 기본 오류 메시지를 표시합니다.

@jjjsun
jjjsun requested review from Seojegyeong and YermIm August 10, 2026 04:38
@jjjsun jjjsun self-assigned this Aug 10, 2026
@jjjsun jjjsun added the 🔨 Refactor 코드 리팩토링 label Aug 10, 2026
@coderabbitai

coderabbitai Bot commented Aug 10, 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: 6c6de92e-6e03-43ea-a3cc-3f9a94196495

📥 Commits

Reviewing files that changed from the base of the PR and between 5588001 and d20b289.

📒 Files selected for processing (1)
  • src/hooks/auth/useDeleteMyAccount.ts

📝 Walkthrough

Walkthrough

회원 탈퇴 오류 처리에서 오류 코드별 WITHDRAW_ERROR_MESSAGES 매핑을 제거했습니다. API 응답의 message를 우선 사용하며, 메시지가 없으면 기존 기본 오류 메시지를 사용합니다.

Changes

회원 탈퇴 오류 처리

Layer / File(s) Summary
API 오류 메시지 사용
src/hooks/auth/useDeleteMyAccount.ts
오류 코드별 메시지 매핑을 제거했습니다. apiError.message를 우선 사용하고, 값이 없으면 기본 오류 메시지를 사용합니다.

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

Possibly related PRs

Suggested reviewers: yermim, seojegyeong

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 회원 탈퇴 에러 메시지 매핑 제거라는 핵심 변경을 명확하고 간결하게 설명합니다.
Description check ✅ Passed 관련 이슈, 변경사항, 작업 내용, 미완성 작업, 참고 사항을 모두 작성했고 변경 목적도 설명했습니다.
Linked Issues check ✅ Passed #434의 요구사항인 매핑 제거와 apiError.message 직접 사용을 모두 반영했습니다.
Out of Scope Changes check ✅ Passed 변경 범위가 회원 탈퇴 에러 처리의 메시지 매핑 제거와 API 메시지 사용으로 제한되어 있습니다.
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 refactor/#434

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.

@github-actions

Copy link
Copy Markdown

📚 Storybook 배포 완료

항목 링크
📖 Storybook https://69a147b60a56365d9e2185ef-mknygvvmyd.chromatic.com/
🔍 Chromatic https://www.chromatic.com/build?appId=69a147b60a56365d9e2185ef&number=602

@YermIm YermIm 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.

P4: 확인했습니다!

@jjjsun
jjjsun merged commit b7126dc into develop Aug 10, 2026
3 checks passed
@jjjsun
jjjsun deleted the refactor/#434 branch August 10, 2026 05:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 Refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

♻️ [Refactor] 회원 탈퇴 에러 메세지 매핑 제거

2 participants