Skip to content

[♻️ Refactor/#394] network, http 에러 분리#395

Merged
Lseojeong merged 2 commits into
devfrom
refactor/#394
May 26, 2026
Merged

[♻️ Refactor/#394] network, http 에러 분리#395
Lseojeong merged 2 commits into
devfrom
refactor/#394

Conversation

@Lseojeong

@Lseojeong Lseojeong commented May 26, 2026

Copy link
Copy Markdown
Owner

#️⃣연관된 이슈

체크 사항

  • UI 동작 및 레이아웃 확인
  • 콘솔 로그/에러 없음
  • 기능 정상 동작
  • 팀 컨벤션에 맞게 구현했는지

📝작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요(이미지 첨부 가능)

  • 인증 에러 처리를 HttpError와 NetworkError로 분리해 HTTP 응답 에러와 네트워크 실패를 명확하게 구분했습니다.
  • 로그인/회원가입에서 4xx는 기존처럼 폼 에러로 처리하고, 500대 서버 에러만 error.tsx로 이동하도록 정리했습니다.

스크린샷 (선택)

추가한 라이브러리 (선택)

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@Lseojeong Lseojeong self-assigned this May 26, 2026
@Lseojeong Lseojeong added the ♻️ Refactor 코드 리팩토링 label May 26, 2026
@Lseojeong Lseojeong linked an issue May 26, 2026 that may be closed by this pull request
1 task
@vercel

vercel Bot commented May 26, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
bookscape Ready Ready Preview, Comment May 26, 2026 10:45am

@gemini-code-assist

Copy link
Copy Markdown

Summary of Changes

Hello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이번 PR은 기존의 통합된 에러 처리 방식을 개선하기 위해 ApiError를 HttpError와 NetworkError로 분리하는 리팩토링을 수행했습니다. 이를 통해 네트워크 통신 실패와 서버 응답 에러를 명확히 구분하여 처리할 수 있게 되었으며, 특히 인증 관련 페이지에서 서버 에러에 대한 대응을 강화하여 안정성을 높였습니다.

Highlights

  • 에러 클래스 분리: 기존의 ApiError 클래스를 HttpError로 변경하고, 네트워크 통신 실패를 처리하기 위한 NetworkError 클래스를 새롭게 추가했습니다.
  • 에러 처리 로직 개선: coreFetch 및 API 호출 코드에서 HTTP 응답 에러와 네트워크 에러를 명확하게 구분하여 처리하도록 로직을 수정했습니다.
  • 로그인/회원가입 에러 대응 강화: 로그인 및 회원가입 과정에서 500번대 서버 에러 발생 시 fatalError 상태를 통해 error.tsx로 이동하도록 개선하여 사용자 경험을 향상시켰습니다.
New Features

🧠 You can now enable Memory (public preview) to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize the Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counterproductive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

@github-actions

Copy link
Copy Markdown

🎨 스토리북 배포 링크 확인

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

이번 풀리퀘스트에서는 기존 ApiErrorHttpError로 이름을 변경하고, 네트워크 장애 상황을 처리하기 위한 NetworkError 클래스를 새롭게 추가하였습니다. 이에 따라 프로젝트 전반의 API 호출 및 에러 핸들링 로직이 수정되었습니다. 리뷰어 피드백으로는 SignupClient.tsx 파일에서 HttpError 처리 시 return문이 누락되어 하단의 토스트 메시지가 중복으로 발생하는 문제가 지적되었습니다. 해당 피드백은 폼 에러와 토스트 에러가 동시에 노출되는 사용자 경험 문제를 방지하기 위한 적절한 개선 사항이므로 반영을 권장합니다.

Comment thread src/features/auth/ui/SignupClient.tsx
@Lseojeong Lseojeong merged commit 3b7d109 into dev May 26, 2026
2 checks passed
@Lseojeong Lseojeong deleted the refactor/#394 branch May 26, 2026 13:14
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] 500 서버 에러 처리

1 participant