Thanks for fixing it 🙏

what if we did something like

	if (query.isError) {
		return <ErrorAlert error={query.error} />;
	}

	if (!query.data) {
		return <Loader />;
	}

so that we don't need to use the ? operator to read from data after the check?

what if we did something like

	if (query.isError) {
		return <ErrorAlert error={query.error} />;
	}

	if (!query.data) {
		return <Loader />;
	}

so that we don't need to use the ? operator to read from data after the check?

I suppose that works but looking at the react query docs, it feels like using isLoading/isPending is the straight-forward clearest way to do things, https://tanstack.com/query/latest/docs/framework/react/guides/queries

but they don't give us the type safety that checking data directly does. I really prefer narrowing the type rather than using ?. everywhere and assuming it'll be fine.