Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

errorCode와 instance로 에러메시지 변경 #1002

Open
wants to merge 2 commits into
base: dev/fe
Choose a base branch
from
Open

errorCode와 instance로 에러메시지 변경 #1002

wants to merge 2 commits into from

Conversation

Jaymyong66
Copy link
Contributor

@Jaymyong66 Jaymyong66 commented Jan 1, 2025 •
edited
Loading

⚡️ 관련 이슈

📍주요 변경 사항

  1. 사용자에게 보여줄 에러메시지를 errorCode와 instance를 통해 구체화시키려 했어요.
  • 하지만 각 에러코드가 스웨거와 너무 다르기도 하고 제대로 적용이 안되어있는거 같아서 이 부분 확인 필요합니다.
  • 프론트 상에서 막혀있는 부분, 예를 들어 아이디 입력 없음 에러나 각 요청에 글자 수 오류 등은 일단 추가하지 않았어요. 개발자도구를 열어서 로그인 버튼의 disabled를 풀어도 내부에서 유효성 검사를 하기에 버튼이 눌리지 않더라구요. 그래서 사용되지 않을거 같은데 두 분은 모든 부분을 추가해야한다고 생각하는지 의견 듣고 싶습니다.
  1. 따라서 사용자에게 보여줄 메시지를 errorCode와 instance를 통해 구분하지 않을 것이라면, errorMessage를 기본적으로 백엔드에서 주고 있는 error detail로 넣어주도록 바꾸었습니다.

🎸기타

아직 논의가 필요한 부분이 많아 draft로 올립니당

🍗 PR 첫 리뷰 마감 기한

1/3 23:59

@Jaymyong66 Jaymyong66 added feature 기능 추가 FE 프론트엔드 labels Jan 1, 2025
@Jaymyong66 Jaymyong66 added this to the 7차 스프린트 💭 milestone Jan 1, 2025
@Jaymyong66 Jaymyong66 self-assigned this Jan 1, 2025
Jaymyong66
Jaymyong66 commented Jan 1, 2025
View reviewed changes
frontend/src/api/Error/getErrorMessage.ts
Comment on lines +19 to +21
// 이거 멤버 ID 어떻게 가져오지,, 파라미터가 필요할 땐 백엔드의 도움을 받아야하나
// if (errorCode === ERROR_CODE.NOT_FOUND_ENDPOINT && instance === END_POINTS.memberTemplates(memberId)) {
// }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

백엔드에서 주는 에러 instance에 memberId가 포함되어있는데, 이걸 어떻게 매칭시킬까요... 흠

Copy link
Collaborator

Choose a reason for hiding this comment

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

이거 어떤 에러 상황인지,, 알 수 있나욜?

@Jaymyong66 Jaymyong66 marked this pull request as ready for review January 13, 2025 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@healim01 healim01 Awaiting requested review from healim01

@Hain-tain Hain-tain Awaiting requested review from Hain-tain

At least 1 approving review is required to merge this pull request.

Assignees

@Jaymyong66 Jaymyong66

Labels
FE 프론트엔드 feature 기능 추가
Projects
2024-code-zap
Status: Todo
Milestone
7차 스프린트 💭
Development

Successfully merging this pull request may close these issues.
2 participants
@Jaymyong66 @healim01