refactor(useQueries):add types for result #2568
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/tannerlinsley/react-query/HornbqK5qYvN7KPhKsP3N2AQ6rwr |
| import { UseQueryOptions, UseQueryResult } from './types' | ||
|
|
||
| export function useQueries(queries: UseQueryOptions[]): UseQueryResult[] { | ||
| export function useQueries<TData = unknown, TError = unknown>(queries: UseQueryOptions[]): UseQueryResult<TData,TError>[] { |
There was a problem hiding this comment.
I'm afraid I don't think it's that easy. we've had an open PR here that didn't quite make it: #1527
and there is an open issue with lots of discussions here: #1675
one problem is that every query in useQueries can have it's own type. It's not only for fetching N times the same query, I can also do:
useQueries([
{ queryKey: 'foo', queryFn: fetchFoo },
{ queryKey: 'bar', queryFn: fetchBar },
] as const)
where the result type should be ideal a tuple: [UseQueryResult<Foo>, UseQueryResult<Bar>]
There was a problem hiding this comment.
True. I was too hopeful that it would be easy.
Thanks for your time. closing the pr
2 participants
Adds types for useQueries beacus previously it was not possible to type the result of useQueries