[FE] 서버에서 보내주는 에러메시지 활용하도록 수정(#544) #546
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
pp449
approved these changes
Oct 10, 2024
| @@ -36,7 +36,7 @@ const useMutateAuth = () => { | |||
| queryClient.invalidateQueries({ queryKey: [QUERY_KEYS.OPENED_ROOM_LIST] }); | |||
| queryClient.invalidateQueries({ queryKey: [QUERY_KEYS.CLOSED_ROOM_LIST] }); | |||
| localStorage.clear(); | |||
| navigate("/"); | |||
| window.location.replace("/"); | |||
There was a problem hiding this comment.
메인 페이지에서 로그아웃 했을 때 navigate("/") 로 하니까 참여중 탭이 그대로 보였습니다. 네이버의 로그아웃 레퍼런스를 참고하여 우리 서비스도 로그아웃 시에 강제로 "/" 페이지로 새로고침 되게 하였습니다.
이러한 이유 땜에 강제 새로고침을 하는 것으로 바꿨습니다. 다른 의견 있으시면 바로 말씀해주세요!!😊
00kang
approved these changes
Oct 10, 2024
| endpoint: API_ENDPOINTS.REVIEW_COMPLETE, | ||
| body: { | ||
| roomId, | ||
| revieweeId, | ||
| }, | ||
| errorMessage: MESSAGES.ERROR.POST_PARTICIPATE_IN, | ||
| errorMessage: MESSAGES.ERROR.POST_REVIEW_COMPLETE, |
There was a problem hiding this comment.
ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ코드 리뷰에서도 놓쳤던....
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📓 스토리북 링크
바로가기
📌 관련 이슈
✨ PR 세부 내용
코드리뷰완료 요청 실패했을 때 상수화한 에러메세지가 없어서 추가했습니다.
메인 페이지에서 로그아웃 했을 때
navigate("/")로 하니까 참여중 탭이 그대로 보였습니다. 네이버의 로그아웃 레퍼런스를 참고하여 우리 서비스도 로그아웃 시에 강제로 "/" 페이지로 새로고침 되게 하였습니다.errorMessage를 인자로 그대로 받지만
(data.message || errorMessage)로직을 추가하여 배겐드가 넘겨준 에러메세지가 우선적으로 보여지게 하였습니다. 오류가 떴지만 배겐드가 넘겨준 메세지가 null 값이거나 undefined 이면 프론트가 상수화한 에러메세지가 출력됩니다.