frintjs/frint

frint-react: Move subscription to componentDidMount in `observe` HoC

Open

#417 aperta il 11 apr 2018

Vedi su GitHub
 (1 commento) (1 reazione) (0 assegnatari)JavaScript (37 fork)batch import
help wanted

Metriche repository

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

Descrizione

At the moment, it's happening in componentWillMount, which is not recommended.

Guida contributor

Direzione di ricerca
Il problema richiede di spostare la sottoscrizione da componentWillMount a componentDidMount nel componente di ordine superiore `observe`. Poiché componentWillMount è deprecato nelle versioni più recenti di React, si tratta di una migrazione diretta. Cerca il file del componente di ordine superiore pertinente nel pacchetto frint react, individua dove viene utilizzato componentWillMount e sposta la logica di sottoscrizione in componentDidMount. Assicurati di aggiornare eventuali test correlati, se presenti.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
basic React lifecycle knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
90