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

Fix: 로그인 되지 않던 이슈 해결 #432

Merged
merged 2 commits into from
Oct 14, 2023
Merged

Fix: 로그인 되지 않던 이슈 해결 #432

merged 2 commits into from
Oct 14, 2023

Conversation

hozzijeong
Copy link
Collaborator

🔥 연관 이슈

🚀 작업 내용

원인

  • 로그인 API method가 POST인데 tanstack-query에서는 useQuery를 사용하고 있었음
  • 따라서 mutation으로 변경해서 API를 전송하도록 요청
  • 그에 따라 useEffect를 통해 login메서드를 실행시킹
  • 하지만 기존 Authorization 페이지에서는 바로 reminder로 이동하는 Navigator를 반환
  • 즉, reminder페이지로 이동하고 로그인을 요청하는 상황이 발생해벌암;;

해결

  • Authorization에서 Navigator를 반환하던 것을 Loading을 반환하도록 함
  • 그리고 Mutation에서 SuccessCallback으로 로그인에 성공했을 경우 reminder로 navigate하도록 설정
  • 반면 실패했을 때는 에러 메세지와 함께 로그인 페이지로 이동하도록 작성함.

💬 리뷰 중점사항

잘 부탁드립니당

@hozzijeong hozzijeong added 🐛 버그 버그가 발생했어요 🍇 프론트엔드 프론트엔드 관련 이슈입니다 labels Oct 13, 2023
@hozzijeong hozzijeong added this to the 6차 스프린트 milestone Oct 13, 2023
@hozzijeong hozzijeong requested review from bassyu and WaiNaat October 13, 2023 05:37
@hozzijeong hozzijeong self-assigned this Oct 13, 2023
@hozzijeong hozzijeong linked an issue Oct 13, 2023 that may be closed by this pull request
로그인이 한번에 되지 않는다. #431
Closed
WaiNaat
WaiNaat approved these changes Oct 13, 2023
View reviewed changes
Copy link
Member

@WaiNaat WaiNaat left a comment

Choose a reason for hiding this comment

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

멋있어요

bassyu
bassyu approved these changes Oct 14, 2023
View reviewed changes
Copy link
Member

@bassyu bassyu left a comment

Choose a reason for hiding this comment

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

멋져요!

@bassyu bassyu merged commit bfa33f9 into develop Oct 14, 2023
2 checks passed
@bassyu bassyu deleted the fix/431-login_bug branch October 14, 2023 13:26
Choi-JJunho pushed a commit that referenced this pull request Oct 20, 2023
@hozzijeong @Choi-JJunho
fix: 로그인 되지 않던 이슈 해결 (#432)
fb50fd8 
* refactor: 렌더링 이후에 api 호출하도록 변경

* refactor: 로그인 했을 경우 Success, Error callback 작성
hozzijeong added a commit that referenced this pull request Oct 20, 2023
@hozzijeong
fix: 로그인 되지 않던 이슈 해결 (#432)
f2a80f9 
* refactor: 렌더링 이후에 api 호출하도록 변경

* refactor: 로그인 했을 경우 Success, Error callback 작성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WaiNaat WaiNaat WaiNaat approved these changes

@bassyu bassyu bassyu approved these changes

Assignees

@hozzijeong hozzijeong

Labels
🍇 프론트엔드 프론트엔드 관련 이슈입니다 🐛 버그 버그가 발생했어요
Projects
2024-pium
Status: Done
Milestone
6차 스프린트
Development

Successfully merging this pull request may close these issues.

로그인이 한번에 되지 않는다.
3 participants
@hozzijeong @bassyu @WaiNaat