Canceling a Piped upgrade leaves the previous Piped selected

Aperta Adatta ai principianti
  • Qualcosa non va?
#7,620 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
83/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
react, typescript
Ambito
frontend, testing

Direzione di ricerca

Inizia da web/src/components/settings-page/piped/components/upgrade-dialog/index.tsx, in particolare dal gestore di chiusura collegato e dal pulsante Cancel, poi esamina il test Jest di SettingsPipedPage menzionato nell’issue. Riproduci i passaggi con il filtro Active Status e controlla quali ID Piped raggiungono l’API di aggiornamento simulata. Il lavoro è completato quando l’annullamento cancella la selezione e, alla riapertura, è necessaria una nuova selezione prima che Upgrade sia abilitato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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

  1. In Settings → Piped, open Upgrade with an enabled Piped A and a disabled Piped B in the project.
  2. Check Piped A and click Cancel.
  3. Set the Active Status filter to Disabled, reopen Upgrade, and choose a version.
  4. 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.

Lingua principale
Go
Stelle
1.4k
Fork
391
Merge medio
4g 1h
PR unite (30g)
58

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di pipe-cd/pipecd

Tutte le issue di pipe-cd/pipecd

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.