react-navigation / react-navigation/react-navigation.github.io
New redux warning?
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 324
- Forks
- 2k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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?
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
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.
Piste de recherche
Commencez par l’avertissement concernant l’intégration de Redux et la documentation liée navigating-without-navigation-prop. Examinez les questions de l’issue et la formulation existante, puis clarifiez pourquoi l’intégration est déconseillée et identifiez l’approche recommandée pour la navigation après une action ; la documentation doit répondre directement aux deux questions.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, react, react-native, redux
- Domaine
- documentation
- Type d'issue
- Documentation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100