Canceling a Piped upgrade leaves the previous Piped selected
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 83/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- react, typescript
Línea de trabajo
Empieza con web/src/components/settings-page/piped/components/upgrade-dialog/index.tsx, especialmente con el controlador de cierre asociado y el botón Cancel, y luego inspecciona la prueba Jest de SettingsPipedPage mencionada en el issue. Reproduce los pasos con el filtro Active Status y comprueba qué IDs de Piped llegan a la API de actualización simulada. El trabajo está terminado cuando cancelar borra la selección y, al volver a abrir, se requiere una nueva selección antes de que Upgrade esté habilitado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happened
Clicking Cancel in the Piped upgrade dialog keeps the checked Piped IDs. If the Active Status filter changes before the dialog reopens, a previously selected Piped can disappear from the table and still be included in the next upgrade request.
The footer's Cancel button bypasses handleClose, so upgradePipedIds survives when the parent hides the dialog. Close handler · Cancel button
What you expected to happen
Cancel clears the selected Pipeds. Reopening the dialog requires a new selection before Upgrade is enabled.
How to reproduce it
- In Settings → Piped, open Upgrade with an enabled Piped A and a disabled Piped B in the project.
- Check Piped A and click Cancel.
- Set the Active Status filter to Disabled, reopen Upgrade, and choose a version.
- The visible checkbox for Piped B is clear, but Upgrade is enabled.
A local Jest test ran these steps on SettingsPipedPage. Clicking Upgrade passed pipedIdsList: ["piped-a"] to the mocked update API.
Environment
Web console source at 87c6da5e7 on master.
- Lenguaje dominante
- Go
- Estrellas
- 1.4k
- Forks
- 391
- Merge medio
- 4 d 1 h
- PR fusionados (30 d)
- 58
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 pipe-cd/pipecd
-
Applications at the repository root are not triggered by new commitsPosiblemente ocupada @Daksha1611 la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 2 días
-
pipedv1 kubernetes plugins ignore trafficRouting.istio.host in K8S_TRAFFIC_ROUTINGPosiblemente ocupada @Dhirenderchoudhary la tomó hoy. Abierto
Dificultad 2/5 Medio día Aptitud para principiantes 74/100
pipe-cd/pipecd#7626 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 2 días
-
pipectl transfer restore application drops backed-up labelsPosiblemente ocupada @bhavishyajain545 la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
pipe-cd/pipecd#7622 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 2 días
Todos los issues de pipe-cd/pipecd
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
docker-compose: postgres healthcheck should probe TCP (pg_isready -h localhost)Posiblemente ocupada @zezo-mohamed22 la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
OneBusAway/watchdog#305 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
self-care self-care:agent-discoverability
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
githubnext/gh-cao#20928 ·
Los mantenedores suelen responder en 1 día
-
automation documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
SocialGouv/iterion#3287 ·
Los mantenedores suelen responder en 1 día