jamesplease/react-request
change `fetching` to be set to `true` on mount for non-lazy requests
開放
#178 建立於 2018年5月25日
breakingenhancementhelp wanted
倉庫指標
- 星標
- (358 顆星)
- PR 合併指標
- (30 天內沒有已合併 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?