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.

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

[GGFE-83] 메인 403에러 alert 추가 #861

Merged
merged 2 commits into from
Jun 22, 2023

Conversation

42sungwook
Copy link
Contributor

@42sungwook 42sungwook commented Jun 21, 2023

📌 개요

  • 메인에서 카카오 로그인으로 인한 에러일시 error 페이지가 아닌 alert 문구를 띄우게 수정

💻 작업사항

  • JH05/JH02 error 일때 403 (권한) 에러면 alert 문구를 띄우게 했습니다!

✅ 변경로직

@42sungwook 42sungwook self-assigned this Jun 21, 2023
Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

에러 페이지로 이동되지 않는 것 확인했습니다! 👍

근데 프로필 수정에서 403 에러는 뜨는데 취소를 누르면 아래처럼 입력한 내용이 반영되고 있어요 (새로고침했을때 사라지는걸 봐서는 DB에 저장되는건 아닌거같고 프론트 랜더링 문제인 것 같기도 하네여.... 🤔)

Screen_Shot 2023-06-22 15 02 12

그리고 프로필 수정이나 경기 등록을 하려면 42계정과 연동해야 하는 거니까
alert 메시지를 "42 계정과 연동하지 않은 유저" 라고 안내하는 것도 괜찮을 것 같아요 😆

@42sungwook
Copy link
Contributor Author

에러 페이지로 이동되지 않는 것 확인했습니다! 👍

근데 프로필 수정에서 403 에러는 뜨는데 취소를 누르면 아래처럼 입력한 내용이 반영되고 있어요 (새로고침했을때 사라지는걸 봐서는 DB에 저장되는건 아닌거같고 프론트 랜더링 문제인 것 같기도 하네여.... 🤔)

Screen_Shot 2023-06-22 15 02 12 그리고 프로필 수정이나 경기 등록을 하려면 42계정과 연동해야 하는 거니까 alert 메시지를 "42 계정과 연동하지 않은 유저" 라고 안내하는 것도 괜찮을 것 같아요 😆

해당 부분 요청 성공했을 때만 recoil 상태 업데이트되도록 고쳤습니다~

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

오 이제 안되는 것 확인했습니다! 수고하셨습니당 👍

@42sungwook 42sungwook merged commit ab14c46 into GGFE-00/Feat/api-refactoring Jun 22, 2023
@42sungwook 42sungwook deleted the GGFE-83-main-403-alert branch June 26, 2023 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants