JSON API mixes null and empty strings for missing URLs
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Comienza rastreando la serialización de la API para list_archive_url, webscm_url y los demás campos de URL nullable mencionados en el issue. Determina si el comportamiento entre null y vacío está controlado por DRF o Patchwork y, a continuación, verifica que las URL nullable produzcan null de forma coherente y que este comportamiento esté cubierto por las pruebas de API relevantes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 317
- Forks
- 91
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de getpatchwork/patchwork
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
getpatchwork/patchwork#668 ·
-
bug web-ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
getpatchwork/patchwork#546 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
getpatchwork/patchwork#664 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
getpatchwork/patchwork#663 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
getpatchwork/patchwork#645 ·
Todos los issues de getpatchwork/patchwork
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
learningequality/ricecooker#747 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
BSData/horus-heresy-3rd-edition#3171 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
run-llama/llama_index#23199 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
KhronosGroup/glTF-Blender-IO#2769 ·