A news link can't be cleared, and settings can't be saved without one
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 76/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- backend
Direzione di ricerca
Inizia tracciando l’handler PATCH /settings e la convalida di newsLink, in particolare la chiamata a sanitiseAndValidateUrl; esamina il codice della pagina delle impostazioni che invia newsLink come null quando è vuoto. Controlla i test di convalida esistenti e aggiungi test per i valori vuoti e non vuoti. Il lavoro è completato quando un newsLink vuoto viene accettato come null, i valori non vuoti continuano a essere controllati e le impostazioni possono essere salvate senza un link alle notizie.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- TypeScript
- Stelle
- 4
- Fork
- 0
- Merge medio
- 14h 50m
- PR unite (30g)
- 5
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Garvan-Data-Science-Platform/ctrl
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 82/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 commento ·
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
-
backend bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
backend bug enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 commento ·
Tutte le issue di Garvan-Data-Science-Platform/ctrl
Issue simili
-
ui(ListToolbar): search takes the full row on a phone instead of sharing it with the result countAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
I maintainer di solito rispondono entro 1 giorno
-
priority:high
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
immich-app/immich#32426 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue hacktoberfest help wanted
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
libredb/libredb-studio#1756 ·
I maintainer di solito rispondono entro 1 giorno