jamesplease/react-request

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

Open

#178 ouverte le 25 mai 2018

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (21 forks)github user discovery
breakingenhancementhelp wanted

Métriques du dépôt

Stars
 (358 stars)
Métriques de merge PR
 (Métriques PR en attente)

Description

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?

Guide contributeur

Direction de recherche
Localisez l'implémentation du composant Fetch et modifiez l'état initial pour définir fetching sur true au montage lorsque lazy est false.
Stack technique
javascriptreact
Domaine
frontend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
React
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75