jamesplease/react-request
在 GitHub 查看change `fetching` to be set to `true` on mount for non-lazy requests
Open
#178 创建于 2018年5月25日
breakingenhancementhelp wanted
仓库指标
- Star
- (358 star)
- PR 合并指标
- (PR 指标待抓取)
描述
I was wondering if you'd consider a slight change for non-lazy behavior. Right now it seems like you need to do:
<Fetch url="/me">
{({ fetching, failed, data }) => {
if (fetching) return ...
if (failed) return ...
if (!data) return null
return ...
}}
</Fetch>
There's an extra case where the component has just mounted, but hasn't yet started the request where fetching == false but data == null. Apollo seems to avoid this, such that you can handle the fetching and failed cases up front, and then assume you have data after that, which just reduces a little of the boilerplate.
Would you be down for that change?