woowacourse-teams / 2024-cruru

복잡했던 리크루팅을 단 하루만에 크루루.
https://www.cruru.kr
18 stars 6 forks source link

refactor-fe: Tanstack Query 데이터 캐싱 정책 변경 #945

Closed lurgi closed 2 weeks ago

lurgi commented 2 weeks ago

목적

데모데이때 발생한 에러는 다음과 같습니다.

원인은 다음과 같습니다.

해당 문제를 해결하기 위한 방법으로 데모데이 때 발생한 문제를 해결한 방법은 다음과 같습니다.

이전 방법은 Tanstack Query의 캐싱 기능을 사용하지 못한다는 문제가 생깁니다. 이는 Tanstack Query의 강력한 기능을 사용하지 않는 것이라 생각하고, 렌더링에서의 사용성 역시 떨어뜨리는 문제라 판단했습니다. 따라서 다음과 같은 문제 해결방법을 생각했습니다.

  1. 로그인, 자동 로그인시 동적 라우팅으로 /dashboard 라우트로 이동하는 코드를 정적 라우팅으로 변경한다. 해당 방법을 통해 문제를 해결할 수 있었지만, 기존 로그인 페이지에서 이미 가지고 있는 리소스를 재요청하는 문제가 있었습니다.
  2. 에러 페이지에서 tanstack Query에서 사용하고 있는 데이터를 clear해준다. <- 해당 방법을 선택하여 구현하는 것을 제안합니다.

작업 세부사항

참고 사항

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


REFAC_TANSTACK_CACHE

github-actions[bot] commented 2 weeks ago

🚀 안녕하세요 @lurgi님! 작업을 시작하셨군요?

작업하실 브랜치를 컨벤션에 맞게 현재 이슈 ✨refactor-fe: Tanstack Query 데이터 캐싱 정책 변경✨에 연결하여 생성헤드렸어요! 생성된 브랜치명: 945-fe-FAC_TANSTACK_CACHE

해당 브랜치가 연결된 ⚡️Draft PR⚡️ 생성되었으니 확인해보세요

작업이 완료된 후, 리뷰어를 선택하고 Draft PR 내부의 Ready for review 버튼을 눌러주시면 됩니다! :) 제가 Slack으로 리뷰어에게 DM으로 알려드릴게요

github-actions[bot] commented 2 weeks ago

Could not create PR (Validation Failed: {"resource":"PullRequest","field":"base","code":"invalid"} - https://docs.github.com/rest/pulls/pulls#create-a-pull-request)