Calling setState in componentDidUpdate
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 25/100
- Type d'issue
- Fonctionnalité
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- eslint, javascript, react
Piste de recherche
Commencez par la documentation liée de la règle react/no-did-update-set-state et par la documentation de React concernant componentDidUpdate. Déterminez quelle modification de la règle ou de ses indications l’issue propose, puis vérifiez le comportement attendu au regard des réserves documentées ; le travail est terminé lorsqu’un périmètre a été défini et que la mise à jour correspondante du projet a été effectuée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
It might be worth revisiting react/no-did-update-set-state.
Now that componentWillReceiveProps() is being deprecated, componentDidUpdate() is the only "safe" way to detect changes in props and set state accordingly.
The React docs say calling setState() in componentDidUpdate() is permitted, albeit with caveats.
You may call setState() immediately in componentDidUpdate() but note that it must be wrapped in a condition like in the example above, or you’ll cause an infinite loop. It would also cause an extra re-rendering which, while not visible to the user, can affect the component performance.
- Langage dominant
- JavaScript
- Étoiles
- 148k
- Forks
- 26.5k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedPeut-être pris @bodapatisaikrishna l’a pris il y a 40 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
airbnb/javascript#3237 · 6 commentaires ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Peut-être pris @Developer-shivamMishra l’a pris il y a 31 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
airbnb/javascript#3152 · 4 commentaires ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Peut-être pris @dataCenter430 l’a pris il y a 235 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 48/100
airbnb/javascript#3238 · 8 commentaires ·
-
Upgrading eslint-plugin-react-hooksPeut-être pris @weihongyu12 l’a pris il y a 381 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
airbnb/javascript#3186 · 3 commentaires · 2 réactions ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
airbnb/javascript#3173 · 9 commentaires · 1 réaction ·
Toutes les issues de airbnb/javascript
Issues similaires
-
[version] Windows WMPF 26042 偏移Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
evi0s/WMPFDebugger#303 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
HakanSeven12/OpenCADStudio#2257 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
[bug] extension mappingOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
-
bug good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
-
bug code-annotation
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
quarto-dev/quarto-cli#15215 ·
Les mainteneurs répondent en général sous 1 jour