Navigate doesn't clear search params
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- react, typescript
- Área
- frontend
Línea de trabajo
Start at the navigate('/foo') entry point described in the issue and trace how the destination URL is assembled when no replacement search string is provided. Done means navigation to another path clears the existing search parameters by default while an explicitly supplied search string still works; the payload names no files or tests to run.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Calling navigate('/foo') will leave search params in place unless there's a replacement search string after the /foo. I don't know why I'd want search to persist between different pages. And if I did, I could always do it manually with navigate(`/foo?${search}`).
This caused me issues where I have a couple routes that use similarly named search params, but whose values absolutely shouldn't carry over. So now I have to do navigate('/foo?dummy') in all of my navigation logic to prevent this behavior.
This really shouldn't be the default that people have to fight with.
- Lenguaje dominante
- TypeScript
- Estrellas
- 7.9k
- Forks
- 193
- Merge medio
- 9 min
- PR fusionados (30 d)
- 1
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de molefrog/wouter
-
Wouter double-decodes search params in `useSearchParams`, which can lead to �Posiblemente ocupada @priyas1672 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
-
Links using ~ do not respect `base`Posiblemente ocupada @molefrog la tomó hace 25 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
Todos los issues de molefrog/wouter
Issues similares
-
bug ready-for-agent
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
PhyberApex/kuroshiro#1687 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
microsoft/monaco-editor#5563 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
awslabs/visual-asset-management-system#937 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100