react-navigation / react-navigation/react-navigation.github.io
New redux warning?
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 324
- Forks
- 2k
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Hello!
We used react-navigation on a project a while ago, and integrated it with our Redux store as per the documentation. But I've just noticed you now have a warning telling people not to do this.
- Can you elaborate why it's bad? "You probably don't need to do this" is a bit ambiguous.
- Managing navigation through actions was our main reason for doing it; it meant one
LOGIN_SUCCEEDEDaction would mutate both our app state and the navigation state to put the user in the right place. What's the recommended approach instead? Is it to navigate without the navigation prop?
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit dem Warnhinweis zur Redux-Integration und der verlinkten Dokumentation navigating-without-navigation-prop. Prüfe die Fragen des Issues und die bestehende Formulierung. Kläre anschließend, warum von der Integration abgeraten wird, und ermittle den empfohlenen Ansatz für die Navigation nach einer Aktion; die Dokumentation sollte beide Fragen direkt beantworten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, react, react-native, redux
- Bereich
- documentation
- Issue-Typ
- Dokumentation
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100