Ordering '/api/projects' with 'list_id' or 'list_name' doesn't work
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
Línea de trabajo
Comienza en patchwork/api/project.py, en ProjectList, e inspecciona los ordering_fields utilizados por /api/projects. Reproduce la ordenación por list_id y list_name y verifica después que el endpoint ya no genere el FieldError indicado para las solicitudes ascendentes y descendentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
If you set the ordering filter to list_id ascending/descending or list_name ascending/descending in /api/projects, the following exception will be thrown:
FieldError at /api/projects/
Cannot resolve keyword 'link_name' into field. Choices are: bundle, delegationrule, id, linkname, listemail, listid, maintainer_project, name, patch, scm_url, send_notifications, series, subject_match, submission, use_tags, web_url, webscm_url
Suggested fix:
diff --git a/patchwork/api/project.py b/patchwork/api/project.py
index 3609f73..c249f54 100644
--- a/patchwork/api/project.py
+++ b/patchwork/api/project.py
@@ -83,7 +83,7 @@ class ProjectList(ProjectMixin, ListAPIView):
search_fields = ('link_name', 'list_id', 'list_email', 'web_url',
'scm_url', 'webscm_url')
- ordering_fields = ('id', 'name', 'link_name', 'list_id')
+ ordering_fields = ('id', 'name', 'linkname', 'listid')
ordering = 'id'
Does this fix look good enough? Should I send a patch?
- 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
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
canonical/paas-charm#368 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
tech debt
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
-
addition to tracking list Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
StevenBlack/hosts#3256 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
qualcomm/qai-appbuilder#275 ·