atom/github

Replace getDerivedStateFromProps with memoization

Open

#1.611 geöffnet am 26. Juli 2018

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (1.069 Stars) (469 Forks)batch import
code-cleanup :wrench:good first issue

Beschreibung

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

Contributor Guide

Tech Stack
javascriptreact
Domain
frontend
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Basic React knowledgeUnderstanding of memoization
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
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.