-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: Next.js app 디렉토리 기반 @tanstack/react-query 설정 #7
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
95cf998
to
f5edbbe
Compare
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.
민수님 생각이 궁금해요
retry: false, | ||
refetchOnMount: false, | ||
refetchOnWindowFocus: false, | ||
suspense: true, |
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.
@alstn2468
https://tanstack.com/query/v4/docs/react/community/suspensive-react-query
@lukemorales/query-key-factory 이 라이브러리도 쓰신다면 전역에 suspense를 기본 옵션으로 사용할 거라면 @suspensive/react-query는 어때요? 타입적으로 좀 더 간결하게 사용할 수 있을 걸로 기대되어서요. 어떤 점을 개선해야할 지 알고 싶어요ㅎㅎ
tkdodo님과도 v5에 추가할 때 어떻게 하면 좋을지 트위터에서 같이 이야기하고 alpha버전에서 이 부분 반영할 수 있으면 좋겠는데 여러 의견이 필요해요. 혹시 관심있다면 같이 이야기해보면 좋을 거 같아요.
관련 Pull Request: TanStack/query#5739
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.
오 좋은 의견 감사합니다.
작업 내용