Canceling a Piped upgrade leaves the previous Piped selected
维护者通常 2 天内回复
还没有人认领这个 Issue。
评估
- 难度
- 2/5
- 预计耗时
- 1-3 小时
- 新手友好度
- 83/100
- Issue 类型
- 缺陷
- 描述清晰度
- 描述清楚
- 活跃度
- 活跃
- 技术栈
- react, typescript
调研方向
从 web/src/components/settings-page/piped/components/upgrade-dialog/index.tsx 开始，重点查看关联的关闭处理器和Cancel按钮，然后检查issue中提到的 SettingsPipedPage Jest测试。使用Active Status筛选器重现这些步骤，并检查哪些Piped ID传到了模拟的更新API。完成标准是取消操作会清除选择，并且重新打开后必须重新选择，Upgrade才会启用。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Go
- 星标
- 1.4k
- 派生
- 391
- 平均合并
- 4 天 1 小时
- 30 天内合并 PR
- 58
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
pipe-cd/pipecd 的其他 Issue
-
Applications at the repository root are not triggered by new commits可能已有人在做 @Daksha1611 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 2 天内回复
-
pipedv1 kubernetes plugins ignore trafficRouting.istio.host in K8S_TRAFFIC_ROUTING可能已有人在做 @Dhirenderchoudhary 今天认领。 未关闭
难度 2/5 半天 新手友好度 74/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 2 天内回复
-
pipectl transfer restore application drops backed-up labels可能已有人在做 @bhavishyajain545 于 2 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 86/100
维护者通常 2 天内回复
-
难度 2/5 1-3 小时 新手友好度 83/100
维护者通常 2 天内回复
相似的 Issue
-
needs-triage typo
难度 1/5 1 小时以内 新手友好度 88/100
llm-d-incubation/llm-d-scaling-manager#122 ·
维护者通常 1 天内回复
-
good first issue severity: low tech-debt
难度 2/5 半天 新手友好度 75/100
TMHSDigital/subenum#329 ·
维护者通常 1 天内回复
-
[Bug] Backend summarize min() / max() panics when a field has no numeric values可能已有人在做 @weng5e 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 64/100
grafana/grafana-infinity-datasource#1832 ·
维护者通常 1 天内回复
-
tech-debt
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
GillMacedo/godebugdiff#13 ·