JSON API mixes null and empty strings for missing URLs
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez par suivre la sérialisation de l’API pour list_archive_url, webscm_url et les autres champs URL nullable mentionnés dans l’issue. Déterminez si le comportement entre null et une valeur vide est contrôlé par DRF ou Patchwork, puis vérifiez que les URL nullable produisent systématiquement null et que ce comportement est couvert par les tests API concernés.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
This is in quite a few places.
list_archive_url is null (though it can be "" too! not sure what the correlation is), most other nullable URLs i.e. webscm_url are "".
I believe the best option is to have them all be null, avoiding cases where a JSON parser knows a field should contain a URL, finds a value (since it's not null), and can't parse it as a valid URL.
Also not sure if this is a DRF quirk or something Patchwork can control.
- Langage dominant
- Python
- Étoiles
- 317
- Forks
- 91
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Guide de contribution
Ouvrir 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 getpatchwork/patchwork
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
getpatchwork/patchwork#546 · 2 commentaires ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
getpatchwork/patchwork#664 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
getpatchwork/patchwork#663 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 45/100
getpatchwork/patchwork#645 ·
Toutes les issues de getpatchwork/patchwork
Issues similaires
-
essnmx good first issue
Difficulté 1/5 Moins d'une heure Accessibilité débutants 95/100
-
[Feature] 奇物选择添加优先级 Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Giskard-AI/giskard-oss#2840 · 1 commentaire ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Ouvertearea: repo bug perceived difficulty: 2
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
yeti-platform/yeti#1380 ·