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

feat/router-fix: 라우팅 문제 fix #180

Merged
merged 1 commit into from
Dec 7, 2023
Merged

feat/router-fix: 라우팅 문제 fix #180

merged 1 commit into from
Dec 7, 2023

Conversation

kiuuon
Copy link
Collaborator

@kiuuon kiuuon commented Dec 7, 2023

유저정보를 서버에서 받아오고 라우팅을 막는 과정에서 문제가 있어서 서버에서 유저정보를 받아올때까지 isLoading을 통해 기다려주는 방식으로 일단 해결했습니다.

Checklist

  • Code Review: 작성한 코드를 다시 한 번 꼼꼼이 확인했나요?
  • Testing: 앱이 잘 구동되는지 개발한 기능이 문제 없이 작동하는지 확인했나요?

Description

유저정보를 서버에서 받아오고 라우팅을 막는 과정에서 문제가 있어서 서버에서 유저정보를 받아올때까지 isLoading을 통해 기다려주는 방식으로 일단 해결했습니다.

Changes Made

client/src/contexts/AuthProvider.tsx

  • user 정보를 다 받아와야지만 isLoading을 false로 바꿔줍니다.
    client/src/pages/UserBasedRoute.tsx
  • isLoading이 false일때만 라우팅을 시켜주고 라우팅이 되면 다시 isLoading을 true로 바꿔줍니다.

아직 UserBasedRoute를 통해 라우팅 될 때 유저가 무조건 host가 되는 문제는 해결하지 못했습니다.

Extra Comments

유저정보를 서버에서 받아오고 라우팅을 막는 과정에서 문제가 있어서 서버에서 유저정보를 받아올때까지 isLoading을 통해 기다려주는 방식으로 일단 해결했습니다.
@kiuuon kiuuon requested review from glowisn and Lukaid-dev December 7, 2023 13:43
@kiuuon kiuuon self-assigned this Dec 7, 2023
Copy link
Collaborator

@Lukaid-dev Lukaid-dev left a comment

Choose a reason for hiding this comment

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

LGTM! 고육지책 느낌이지만... 일단은 이렇게 처리하고 나중에 리팩토링 합시다!

@kiuuon kiuuon removed the request for review from glowisn December 7, 2023 13:46
@kiuuon kiuuon merged commit 686fa22 into develop Dec 7, 2023
1 check passed
@sjn0910 sjn0910 deleted the feat/router-fix branch December 7, 2023 14:56
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.

2 participants