A news link can't be cleared, and settings can't be saved without one
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 76/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript
- Domaine
- backend
Piste de recherche
Commence par retracer le handler PATCH /settings et la validation de newsLink, en particulier l’appel à sanitiseAndValidateUrl ; examine le code de la page des paramètres qui envoie newsLink à null lorsqu’il est vide. Vérifie les tests de validation existants et ajoute des tests couvrant les valeurs vides et non vides. Le travail est terminé lorsqu’un newsLink vide est accepté comme null, que les valeurs non vides sont toujours vérifiées et que les paramètres peuvent être enregistrés sans lien vers les actualités.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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.
- Langage dominant
- TypeScript
- Étoiles
- 4
- Forks
- 0
- Merge moyen
- 14 h 50 min
- PR mergées (30 j)
- 5
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Garvan-Data-Science-Platform/ctrl
-
Difficulté 1/5 1-3 heures Accessibilité débutants 82/100
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Garvan-Data-Science-Platform/ctrl#970 · 1 commentaire ·
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
backend bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
-
backend bug enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Garvan-Data-Science-Platform/ctrl#956 · 1 commentaire ·
Toutes les issues de Garvan-Data-Science-Platform/ctrl
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
stellar/stellar-dev-skill#151 ·
Les mainteneurs répondent en général sous 1 jour
-
Trial & Error: guided Blind offer squeezes on phones and sits under the touch toolbar on iPadOuvertearea:arcade game ready-for-agent
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
fderuiter/deruiter.dev#2281 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
scope: data grid type: bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 2 jours
-
fix(frontend): correct VITE_IS_DEVELPOMENT typo in shared/app.ts and align dev environment flagsOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
vicharanashala/ajrasakha#1439 ·
Les mainteneurs répondent en général sous 1 jour
-
Add: TV5Monde AsieOuverteapproved check:passed streams:add
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour