react-navigation / react-navigation/react-navigation.github.io
Incomplete documentation on optional parameters with deep linking and path
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 324
- Forks
- 2k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Current Behavior
According to tutorials and documentation I managed to gather online, this piece of code should allow for an optional parameter param2, but it seems that it doesn't work and the path is never matched.
ex: {
screen: Screen,
},
path: "ex/:param1(/:param2)",
},```
### Expected Behavior
I hoped here that different paths would be matched like:
- app://ex/p1
- app://ex/p1/p2
Both paths should have been matched according to my understanding of the `path` keyword (almost absent from the documentation, hence the issue).
### Your Environment
| software | version
| ---------------- | -------
| react-navigation | 4.0.10
| react-native | 0.61.3
| node | n/a
| npm or yarn | n/a
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza con la documentación sobre deep-linking y los ejemplos de configuración de path descritos en el issue. Comprueba cómo se documentan los parámetros opcionales para react-navigation 4.0.10 y verifica las rutas ex/p1 y ex/p1/p2. Se considera terminado cuando la documentación explica claramente la sintaxis admitida y el comportamiento de coincidencia esperado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, react, react-native
- Área
- documentation, mobile
- Tipo de issue
- Documentación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 32/100