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] React-Query devtools 환경 설정 #463

Merged
merged 2 commits into from
Aug 22, 2024
Merged

[FE] React-Query devtools 환경 설정 #463

merged 2 commits into from
Aug 22, 2024

Conversation

jinhokim98
Copy link
Contributor

issue

구현 사항

react-query devtools 초기 설정을 했습니다.
도입한 목적은 귀여운 야자수를 확인할 수 있기 때문입니다.
사실 devtool을 설치하면 캐싱 데이터의 상태를 눈으로 확인할 수 있어서 셋팅해뒀습니다.
image

논의하고 싶은 부분(선택)

stale time, gc time을 임시로 1분씩 설정했습니다.
추후에 같이 논의해서 수정해봐요

🫡 참고사항

@jinhokim98 jinhokim98 added this to the lev3 milestone Aug 21, 2024
@jinhokim98 jinhokim98 requested review from pakxe, soi-ha and Todari August 21, 2024 12:12
@jinhokim98 jinhokim98 self-assigned this Aug 21, 2024
@jinhokim98 jinhokim98 linked an issue Aug 21, 2024 that may be closed by this pull request
1 task
Copy link
Contributor

@pakxe pakxe left a comment

Choose a reason for hiding this comment

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

리액트 쿼리 정말 잘 아시는 것 같아요. 덕분에 많은 것을 알아갑니다 !

Copy link
Contributor

@soi-ha soi-ha left a comment

Choose a reason for hiding this comment

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

아~ devTools 너무 좋아요..
해야지 생각만 했는데 쿠키가 해줘써.. 쵝오..

Comment on lines +13 to +20
const queryClient = new QueryClient({
defaultOptions: {
queries: {
staleTime: 1000 * 60, // 1 minute
gcTime: 1000 * 60, // 1 minute
},
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

혹시 1분으로 정한 이유가 있나요?
그냥 쿠키가 1분으로 설정한 근거가 궁금했어요~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

stale time을 주지 않으면 invalidateQueries가 의미가 없잖아요. 이거 까먹었다가 dev tools 설치하면서 앗차하고 추가했는데 이거에 대해서 백호랑 웨디랑 잠깐 논의해봤는데 결론이 나진 않긴 했는데;;; 진짜 임시 값이라ㅋㅋㅋ

1분으로 설정한 이유는 사실 큰 이유는 없었는데 그래도 이유를 생각해보자면 관리자가 수정했을 때 사용자가 새로고침 하지 않더라도 1분 뒤에 바로 새 값이 반영되도록 하면 좋지 않을까도 있긴 하네요.

크루위키 할 때 캐싱 시간 12시간 주고 새로고침해도 날라가지 않아서 사용자들이 혼란을 겪은 것도 생각이 났어요. 그래서 수정이 그래도 꽤 일어나는 상황이라면 캐싱 시간을 짧게 가져가 보는 것이 좋을 것 같아서도 있는 것 같아요.

이건 제 생각이라 다른 의견 여러가지 종합해서 적절한 시간을 찾아봐요!

@pakxe pakxe merged commit ebc2bc2 into fe-dev Aug 22, 2024
1 check passed
@pakxe pakxe deleted the feature/#459 branch August 22, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

[FE] React-Query devtools 환경 설정
4 participants