frintjs/frint

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

Aperta

#417 aperta il 11 apr 2018

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

Metriche repository

Star
 (746 stelle)
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
Trova l'HoC observe nel pacchetto frint react, identifica il codice utilizzato per la sottoscrizione in componentWillMount e sposta tale logica in componentDidMount.
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.
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.
85