reactjs / reactjs/react.dev

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

Offen
#632 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
JavaScript
Sterne
11.8k
Forks
7.9k
Ø Merge
1 T. 11 Std.
Gemergte PRs (30 T.)
11

Beschreibung

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

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Öffne content/docs/state-and-lifecycle.md im Abschnitt „State Updates are Merged“ und überprüfe die zitierte Erklärung des flachen Mergings. Aktualisiere die Beschreibung so, dass sie nicht impliziert, dass entweder flaches oder tiefes Merging this.state.posts überschreiben kann; abgeschlossen ist die Aufgabe, wenn die Formulierung das Beispiel korrekt beschreibt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
javascript, react
Bereich
documentation
Issue-Typ
Dokumentation
Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.