reactjs / reactjs/react.dev

Typo in Higher Order Components page

Offen
#3,291 1 Kommentar 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

Hello,

I am new to React so I could be wrong but I think I spotted an issue in this section. Instead of this:

const CommentListWithSubscription = withSubscription(
  CommentList,
  (DataSource) => DataSource.getComments()
);

const BlogPostWithSubscription = withSubscription(
  BlogPost,
  (DataSource, props) => DataSource.getBlogPost(props.id)
);

I believe it should say that:

const CommentListWithSubscription = withSubscription(
  Comment,
  (DataSource) => DataSource.getComments()
);

const BlogPostWithSubscription = withSubscription(
  TextBlock,
  (DataSource, props) => DataSource.getBlogPost(props.id)
);

i.e. the WrappedComponent argument of the withSubscription HOC is wrong in both declarations.

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 die verlinkte Seite zu Higher Order Components und suche den Abschnitt „use HOCs for cross-cutting concerns“. Vergleiche beide withSubscription-Beispiele mit den gemeldeten Komponentennamen und überprüfe anschließend, ob die Dokumentation korrekt gerendert wird; die Aufgabe ist erledigt, wenn die Beispiele die vorgesehenen umschlossenen Komponenten verwenden.

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
52/100

Neue Issues direkt in Ihr Postfach

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