-
Notifications
You must be signed in to change notification settings - Fork 5
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리액트 쿼리 정말 잘 아시는 것 같아요. 덕분에 많은 것을 알아갑니다 !
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아~ devTools 너무 좋아요..
해야지 생각만 했는데 쿠키가 해줘써.. 쵝오..
const queryClient = new QueryClient({ | ||
defaultOptions: { | ||
queries: { | ||
staleTime: 1000 * 60, // 1 minute | ||
gcTime: 1000 * 60, // 1 minute | ||
}, | ||
}, | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
혹시 1분으로 정한 이유가 있나요?
그냥 쿠키가 1분으로 설정한 근거가 궁금했어요~
There was a problem hiding this comment.
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시간 주고 새로고침해도 날라가지 않아서 사용자들이 혼란을 겪은 것도 생각이 났어요. 그래서 수정이 그래도 꽤 일어나는 상황이라면 캐싱 시간을 짧게 가져가 보는 것이 좋을 것 같아서도 있는 것 같아요.
이건 제 생각이라 다른 의견 여러가지 종합해서 적절한 시간을 찾아봐요!
issue
구현 사항
react-query devtools 초기 설정을 했습니다.
도입한 목적은 귀여운 야자수를 확인할 수 있기 때문입니다.
사실 devtool을 설치하면 캐싱 데이터의 상태를 눈으로 확인할 수 있어서 셋팅해뒀습니다.
논의하고 싶은 부분(선택)
stale time, gc time을 임시로 1분씩 설정했습니다.
추후에 같이 논의해서 수정해봐요
🫡 참고사항