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

[FE] fix: 로그인 리다이렉트시 방설정 창이 보이는 오류 해결 #219

Merged
merged 2 commits into from
Dec 7, 2023

Conversation

d0422
Copy link
Collaborator

@d0422 d0422 commented Dec 7, 2023

로그인 리다이렉트시 방설정 창이 보이는 오류 해결

PR 설명

  • 로그인 리다이렉트시 방설정 창이 보이는 오류 해결

✅ 완료한 기능 명세

  • 로그인 리다이렉트시 방설정 창이 보이는 오류 해결

고민과 해결과정

기존의 isSettingDone이 그 역할을 했으나, 현재 isSettingDone는 signal을 시작시키는 트리거형태로 작동하기 때문에, 로그인시의 redirection인지, 첫 접속인지 분기할 필요가 있었다.

hasLogin이라는 변수를 하나 선언하고이전에 저장된 code와 nickName값에 따라 로그인 리다이렉션인지, 처음 접속하는 과정인지를 분기할 수 있도록 했다.

const defaultCode = localStorage.getItem('code');
const defaultNickName = localStorage.getItem('nickName');
const hasLogin = (!!defaultCode || defaultCode === '') && !!defaultNickName;

export default function Room() {
  if (!isConnectionDone && !hasLogin) return <Setting mediaObject={mediaObject} />;
...
}

d0422 added 2 commits December 7, 2023 15:55
로그인 리다이렉트시 방설정 창이 보이는 오류 해결
(#218)
state제거하고 상수형태로 변경
(#218)
@d0422 d0422 requested a review from HBSPS December 7, 2023 07:04
@d0422 d0422 self-assigned this Dec 7, 2023
@d0422 d0422 added FrontEnd 프론트엔드 관련 fix 오류 수정 labels Dec 7, 2023
@d0422 d0422 linked an issue Dec 7, 2023 that may be closed by this pull request
1 task
@d0422 d0422 merged commit 1e67dac into boostcampwm2023:dev Dec 7, 2023
6 checks passed
@d0422 d0422 mentioned this pull request Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 오류 수정 FrontEnd 프론트엔드 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE} 로그인 리다이렉트시 방설정이 뜨는 오류
2 participants