jamesplease/react-request

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

Open

#178 aperta il 25 mag 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (21 fork)github user discovery
breakingenhancementhelp wanted

Metriche repository

Star
 (358 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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?

Guida contributor

Direzione di ricerca
Trova l'implementazione del componente Fetch e modifica lo stato iniziale per impostare fetching su true al mount quando lazy è false.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
React
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75