aksonov/react-native-router-flux

Actions.pop always return true

Open

#3 337 ouverte le 30 oct. 2018

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (8 999 stars) (2 112 forks)batch import
bughelp wanted

Description

Version

Tell us which versions you are using:

  • react-native-router-flux v4.0.4
  • react v16.2.0
  • react-native v0.53.3

Expected behaviour

In RNRF v4.0.0-bate25, Actions.pop return false if the sence is the first page, but 4.0.4 always return true.How do I tell if my stack is currently empty?

Actual behaviour

As above

Guide contributeur

Stack technique
javascriptreact nativereact
Domaine
mobile
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Understanding of React Native navigationFamiliarity with react native router flux APIBasic JavaScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60
Direction de recherche
Investigate the Actions.pop implementation in the library's source code, likely in the core router file. The pop method should check if there is only one scene in the stack and return false in that case. Look for any recent changes that might have removed this check. Test with a minimal React Native app using the library and verify that Actions.pop returns false when navigation stack has only one scene.