`/api/tree` API applies unneeded time filter by default
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 65/100
Línea de trabajo
Comienza siguiendo el endpoint /api/tree y las rutas del dashboard / y /tree, centrándote en dónde recibe sus valores predeterminados interval_in_days. Confirma el comportamiento actual y haz que los valores predeterminados queden sin establecer en estos puntos de entrada; después, verifica que las solicitudes sin el parámetro devuelvan todos los checkouts más recientes disponibles y que la redirección no tenga un intervalo codificado de forma fija.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
/api/tree queries for the latest checkout for each tree. By default, it filters by checkouts in the last 7 days. However, this operation is really fast now (queries a pre-computed table), and there should be no reason to have this limit by default.
Related to this, there is a misagreement between the API and the dashboard here:
- API default is
interval_in_days=7(as mentioned above) - The dashboard root
/redirects to/tree?i=7and queries withinterval_in_days=7(not present in production) - Dashboard
/treequeries withinterval_in_days=30
Proposed solution
Therefore, I suggest:
- Keep
interval_in_daysas optional, but do not filter if unset - Make
/redirect to/treewithout hardcoded parameters - Make
/treequery withinterval_in_daysunset by default- Alternatively, grab the default directly from the backend
- Lenguaje dominante
- Python
- Estrellas
- 10
- Forks
- 32
- Merge medio
- 5 d 22 h
- PR fusionados (30 d)
- 19
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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 kernelci/dashboard
-
4xx and 5xx by endpointPosiblemente ocupada @alanpeixinho la tomó hace 5 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
-
[Local dev environment] PRIVACY.md missing in dashboard containerPosiblemente ocupada @alanpeixinho la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
kernelci/dashboard#2142 · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
[AB Comparison] Add labels to Previous commit and Branch head buttonsPosiblemente ocupada @alanpeixinho la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
kernelci/dashboard#2128 · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 52/100
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
Todos los issues de kernelci/dashboard
Issues similares
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
pytest-dev/pluggy#757 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
NousResearch/hermes-agent#134960 ·
Los mantenedores suelen responder en 1 día
-
HTML backend: `<br>` leaks the internal sentinel U+E000 into list items, headings and captionsPosiblemente ocupada @morten-lagabote la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
docling-project/docling#4671 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
good first issue hacktoberfest infra
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día