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

[Fix] spotlight & reloginDialog error #352

Merged
merged 1 commit into from
Oct 24, 2023
Merged

[Fix] spotlight & reloginDialog error #352

merged 1 commit into from
Oct 24, 2023

Conversation

42sungwook
Copy link
Contributor

@42sungwook 42sungwook commented Oct 24, 2023

Summary

근본적인 해결책을 못찾았습니다.
임시로 spotlight, relogindialog 에러를 고쳤어요

Describe your changes

  • spotlight에서 페이지 이동시 spotlight이 꺼지는 로직을 뺐습니다 -> 언제 쓰이는지 모르겠네요
  • reloginDialog에서 navigate 대신 페이지 새로고침을 시켰습니다 -> 이거는 logout쪽도 이런식으로 바꾸려고 생각중이에요 (atom상태 유지되는 현상 때문)


이것저것 찾아본 결과 Provider 바깥에 Router로 감싸져 있어야 하는 것 같아요
근데 <RouterProvider router={router} /> 형태로 만드셔서
바꾸려면 꽤나 작업이 커질 것 같아 임시 작업만 한 상태입니다
나중에 provider에서 useLocation 또는 navigate를 사용하게 되면 고치긴 해야되는 문제로 보입니다 😢
Ref. 참고사이트

Issue number and link

@42sungwook 42sungwook self-assigned this Oct 24, 2023
@42sungwook 42sungwook linked an issue Oct 24, 2023 that may be closed by this pull request
@42sungwook 42sungwook requested a review from yoopark October 24, 2023 07:35
@yoopark
Copy link
Contributor

yoopark commented Oct 24, 2023

해당 Ref가 6.4 나오기 이전 문서인 것 같아요. navigation 대신 location.href를 사용하는 것이 임시 해결책이 될 수는 있겠네요. 일단 푸시해서 사용하시고 제가 다른 해결책을 찾아보겠습니다.

@42sungwook 42sungwook merged commit 99c710e into develop Oct 24, 2023
8 checks passed
@42sungwook 42sungwook deleted the fix/router branch October 24, 2023 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Router Error
2 participants