jamesplease/react-request

change `fetching` to be set to `true` on mount for non-lazy requests

Open

#178 创建于 2018年5月25日

在 GitHub 查看
 (3 评论) (0 反应) (0 负责人)JavaScript (21 fork)github user discovery
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?

贡献者指南