A news link can't be cleared, and settings can't be saved without one
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 76/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript
- Bereich
- backend
Rechercherichtung
Beginne damit, den PATCH /settings-Handler und die Validierung von newsLink nachzuverfolgen, insbesondere den Aufruf von sanitiseAndValidateUrl; untersuche den Code der Einstellungsseite, der newsLink bei leerem Wert als null sendet. Prüfe die bestehenden Validierungstests und ergänze Tests für leere und nicht leere Werte. Fertig ist die Änderung, wenn ein leeres newsLink als null akzeptiert wird, nicht leere Werte weiterhin geprüft werden und Einstellungen ohne News-Link gespeichert werden können.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 4
- Forks
- 0
- Ø Merge
- 14 Std. 50 Min.
- Gemergte PRs (30 T.)
- 5
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Garvan-Data-Science-Platform/ctrl
-
Schwierigkeit 1/5 1-3 Stunden Anfängerfreundlichkeit 82/100
-
backend bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 Kommentar ·
-
backend bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
backend bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
-
backend bug enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 Kommentar ·
Alle Issues in Garvan-Data-Science-Platform/ctrl
Ähnliche Issues
-
`cf complete zsh >> .zshrc` fails on vanilla zshEvtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
cloudflare/cf#340 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
-
Rename Madson cameraOffen
Schwierigkeit 2/5 Unter einer Stunde Anfängerfreundlichkeit 75/100
StoDevX/AAO-React-Native#8785 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Broken links in the docsOffen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 75/100
-
bnv-manager-v2: accept_invite creates the Mailcow domain admin before the commitEvtl. vergeben @myOmikron hat das heute übernommen. Offenbug ready-for-agent
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100