using select as an onSuccess replacement

[FatahChan]

I am reposting a question from discord

in reference to this article (great article btw)

the recommend way to sync state is to use useEffect in use land

export function useTodos(filters) {
  const { dispatch } = useDispatch()

  const query = useQuery({
    queryKey: ['todos', 'list', { filters }],
    queryFn: () => fetchTodos(filters),
    staleTime: 2 * 60 * 1000,
  })

  React.useEffect(() => {
    if (query.data) {
      dispatch(setTodos(query.data))
    }
  }, [query.data])

  return query
}

now according the docs, select reruns when the data changes assuming data means cached data this means, I ca write something like this

export function useTodos(filters) {
  const { dispatch } = useDispatch()

  const query = useQuery({
    queryKey: ['todos', 'list', { filters }],
    queryFn: () => fetchTodos(filters),
    staleTime: 2 * 60 * 1000,
    select: (data) => {
      if (data) {
        dispatch(setTodos(data))
      }
      return data
    }
  })
  return query
}

is there any drawbacks to this approach?