atom/github

Replace getDerivedStateFromProps with memoization

Open

#1 611 ouverte le 26 juil. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (1 069 stars) (469 forks)batch import
code-cleanup :wrench:good first issue

Description

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

Guide contributeur

Stack technique
javascriptreact
Domaine
frontend
Type d'issue
refactor
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Basic React knowledgeUnderstanding of memoization
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50
Direction de recherche
Search the codebase for components that implement the getDerivedStateFromProps lifecycle method. In each case, evaluate whether the derived state can be computed using memoization with memoize one or by converting to a PureComponent. Follow the guidance in the linked React blog post. Ensure test coverage for the affected components. Since the issue is from 2018, verify that the components still exist and the pattern is still applicable.