atom/github

Replace getDerivedStateFromProps with memoization

Open

#1611 aperta il 26 lug 2018

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

Metriche repository

Star
 (1069 star)
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
Cerca nel codebase i componenti che implementano il metodo del ciclo di vita getDerivedStateFromProps. In ogni caso, valuta se lo stato derivato può essere calcolato utilizzando la memoizzazione con memoize one o convertendo il componente in un PureComponent. Segui le linee guida nel post del blog di React collegato. Assicura la copertura dei test per i componenti interessati. Poiché l'issue è del 2018, verifica che i componenti esistano ancora e che il pattern sia ancora applicabile.
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.
1-3 ore
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 knowledgeUnderstanding of memoization
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50