nfl/react-helmet

onChangeClientState not executing

Open

#328 aperta il 23 ott 2017

Vedi su GitHub
 (2 commenti) (8 reazioni) (0 assegnatari)JavaScript (706 fork)batch import
bughelp wanted

Metriche repository

Star
 (17.167 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

It appears that after v5.0.0 onChangeClientState has some odd behaviour - its seemingly overridden by nested <Helmet /> usage or something else that results in it not executing when there are is more than one use of it. This wasn't the case before and would executed on the component it was specified for as well as all others.

This is useful for running code after Helmet has injected all the specified items - componentDidMount will sometimes return null when querying the DOM for items that should have been injected so we cannot use this.

Guida contributor

Direzione di ricerca
Inizia esaminando il codice sorgente di react helmet, concentrandoti sul metodo componentWillMount e sull'implementazione di onChangeClientState. Controlla il log git per le modifiche tra v4 e v5 per identificare cosa è cambiato. Leggi i due commenti sull'issue per eventuali suggerimenti aggiuntivi. Considera di scrivere una riproduzione minima in CodeSandbox per isolare il bug.
Tech stack
reactjavascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Richiede indagine
Prerequisiti
ReactJavaScriptcomponent lifecycle
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
15