reactjs / reactjs/react.dev

[Documentation] Minor change in description of state-and-lifecycle.md

Ouverte
#632 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Langage dominant
JavaScript
Étoiles
11.8k
Forks
7.9k
Merge moyen
1 j 11 h
PR mergées (30 j)
11

Description

https://github.com/reactjs/reactjs.org/blob/master/content/docs/state-and-lifecycle.md#state-updates-are-merged

The merging is shallow, so this.setState({comments}) leaves this.state.posts intact, but completely replaces this.state.comments.

Type of merging doesn't matter in this case. Neither shallow, nor deep merge can overwrite this.state.posts

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Ouvrez content/docs/state-and-lifecycle.md à la section « State Updates are Merged » et relisez l’explication citée de la fusion superficielle. Mettez à jour la description afin qu’elle n’implique pas que la fusion superficielle ou la fusion profonde puisse écraser this.state.posts ; le travail est terminé lorsque la formulation décrit correctement l’exemple.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
javascript, react
Domaine
documentation
Type d'issue
Documentation
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.