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: 로그인, 파일불러오기 ,파일 삭제 오류 해결 #141

Merged
merged 3 commits into from
Nov 27, 2023

Conversation

d0422
Copy link
Collaborator

@d0422 d0422 commented Nov 27, 2023

PR 설명

로그인, 파일 불러오기, 파일 삭제 오류 해결

✅ 완료한 기능 명세

  • 파일 불러오기에서 모달이 바로 뜨지 않는 문제 해결
  • 파일 삭제가 제대로 되지 않는 문제 해결
  • login next query가 제대로 가지 않던 오류 해결

고민과 해결과정

로그인 버튼 리다이렉트 쿼리 문제

function Main() {
  return (
    <QueryClientProvider client={reactQueryClient}>
      <RouterProvider router={router} />
      <Modals />
    </QueryClientProvider>
  );
}

모달이 Router Provider바깥에서 렌더링 되기때문에 useParams가 제대로 동작하지 않았다.
window.location.pathname으로 얻어와서 처리해주었다.

모달이 바로 뜨지 않는 문제

refetch함수는 promise를 반환하는 비동기 함수다.
await로 해결해주었다.

  const handleLoadCloudCodeFile = async () => {
    await refetch();
    setClick(true);
  };

loginModal next 쿼리파라미터가 제대로 들어가지 않던 이슈 해결
(#140)
delete URL이 잘못 들어간 부분 해결
(#140)
LoadButton refetch에 await가 없어 제대로 모달이 렌더링되지 않던 문제 해결
(#140)
@d0422 d0422 added FrontEnd 프론트엔드 관련 fix 오류 수정 labels Nov 27, 2023
@d0422 d0422 self-assigned this Nov 27, 2023
@d0422 d0422 requested a review from HBSPS November 27, 2023 13:03
@d0422 d0422 merged commit 3d5c755 into boostcampwm2023:dev Nov 27, 2023
6 checks passed
@d0422 d0422 linked an issue Nov 27, 2023 that may be closed by this pull request
3 tasks
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