JSON API mixes null and empty strings for missing URLs
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 35/100
Rechercherichtung
Beginne damit, die API-Serialisierung für list_archive_url, webscm_url und die anderen im Issue erwähnten nullable URL-Felder nachzuverfolgen. Ermittle, ob das Verhalten zwischen null und leer von DRF oder Patchwork gesteuert wird, und überprüfe anschließend, dass nullable URLs konsistent null erzeugen und dass dieses Verhalten durch die relevanten API-Tests abgedeckt ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 317
- Forks
- 91
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
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 getpatchwork/patchwork
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
getpatchwork/patchwork#546 · 2 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100
getpatchwork/patchwork#664 ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
getpatchwork/patchwork#663 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
getpatchwork/patchwork#645 ·
Alle Issues in getpatchwork/patchwork
Ähnliche Issues
-
area: harness bug status: needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Human-Agent-Society/reef#625 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 80/100
learningequality/kolibri#15351 · 2 Kommentare ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Name consistency Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
eellak/triplestore#65 · 1 Kommentar ·