A news link can't be cleared, and settings can't be saved without one
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
Empieza por seguir el flujo del handler PATCH /settings y la validación de newsLink, especialmente la llamada a sanitiseAndValidateUrl; inspecciona el código de la página de configuración que envía newsLink como null cuando está vacío. Revisa las pruebas de validación existentes y añade cobertura para valores vacíos y no vacíos. El trabajo estará terminado cuando se acepte un newsLink vacío como null, se sigan comprobando los valores no vacíos y se puedan guardar los ajustes sin un enlace a noticias.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
PATCH /settings passes every newsLink it receives through sanitiseAndValidateUrl, and @braintree/sanitize-url turns null and '' into about:blank, which fails urlRegex, so the request gets a 422. The settings page always sends newsLink, as null when the field is empty. So an admin can't clear the news link once it's set, and an organisation without one (the schema default) can't save any setting from that page.
The participant portal treats "no news link" as valid (#598): the menu hides News, and /news redirects home. Clearing worked until #862 added the check for #860.
Fix: accept an empty newsLink as null, and only check non-empty values.
Related: #598, #727, #764, #860, #862.
- Lenguaje dominante
- TypeScript
- Estrellas
- 4
- Forks
- 0
- Merge medio
- 14 h 50 min
- PR fusionados (30 d)
- 5
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
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 Garvan-Data-Science-Platform/ctrl
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 82/100
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 comentario ·
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
backend bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
backend bug enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 comentario ·
Todos los issues de Garvan-Data-Science-Platform/ctrl
Issues similares
-
area:plugins enhancement platform:vscode
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
anthropics/claude-code#103166 ·
Los mantenedores suelen responder en 1 día
-
Add: Add pogo tv indiaAbiertocheck:failed streams:add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
iptv-org/iptv#55744 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
jitsi/jitsi-meet#18021 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
spec: discovery input example can contain multiple aliases for one parameter, with no canonical oneAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
x402-foundation/x402#3841 ·
Los mantenedores suelen responder en 1 día