atom/github

Replace getDerivedStateFromProps with memoization

Aperta

#1611 aperta il 26 lug 2018

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (469 fork)batch import
code-cleanup :wrench:good first issue

Metriche repository

Star
 (1069 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Where possible, it would be cleaner and less error-prone to replace components that implement static getDerivedStateFromProps() with PureComponents or memoize-one:

Guida contributor

Direzione di ricerca
Trova tutti i componenti React che utilizzano static getDerivedStateFromProps. Per ogni componente, determina se può essere convertito in un PureComponent o se è appropriata la memoization con memoize one. Segui le indicazioni nel post del blog collegato per sostituire il metodo del ciclo di vita. Esegui i test esistenti per assicurarti che non ci siano regressioni.
Tech stack
reactjavascript
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.
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