jamesplease/react-request

Redundant `render` run with outdated state

Open

#196 aperta il 24 set 2018

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

Metriche repository

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

Descrizione

If I render the same Fetch with new url, first I get it's children rendered with previous state (old url, old data, etc.) It looks like the problem is known: https://github.com/jamesplease/react-request/blob/eca89d5fe79bd546a0641b6443415b722e3f6ec3/src/fetch.js#L107 It feels that it would be simple to fix it by updating Fetch state in componentWillReceiveProps hook. @jamesplease what's your opinion?

Guida contributor

Direzione di ricerca
Implementa `componentWillReceiveProps` nel componente `Fetch` per aggiornare lo stato interno quando la prop `url` cambia, evitando un rendering con stato obsoleto.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
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
ReactJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75