[PR #183] [MERGED] feat: add "Who is using" to HelpMenu with cluster info pre-fill #175

New issue

Closed

opened 2026-02-26 17:37:37 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 17:37:37 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Corsinvest/cv4pve-admin/pull/183
Author: @franklupo
Created: 2/20/2026
Status: ✅ Merged
Merged: 2/20/2026
Merged by: @franklupo

Base: main ← Head: feat/help-menu-who-is-using

📝 Commits (1)

9cc81ac feat: add "Who is using" to HelpMenu with cluster info pre-fill

📊 Changes

7 files changed (+107 additions, -157 deletions)

View changed files

📝 admin.ps1 (+10 -5)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/HelpMenu.razor (+4 -3)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/HelpMenu.razor.cs (+41 -25)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Helpers/ApplicationHelper.cs (+10 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Core/Helpers/PveAdminHelper.cs (+39 -1)
📝 src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/Support.razor (+3 -36)
➖ src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/Support.razor.cs (+0 -86)

📄 Description

Summary

Add "Who is using cv4pve-admin?" entry in HelpMenu that opens a GitHub issue pre-filled with cluster data (nodes, VMs, CPU, RAM, storage) — closes #11
Move GenerateWhoUsingAsync from Support.razor.cs to PveAdminHelper (shared helper)
Refactor HelpMenu to generate all URLs lazily on-click (avoid slow API calls at menu init time)
Add separator --- between multiple clusters in the who-is-using body
Fix admin.ps1 publish: extract Invoke-DockerPrepare so both build docker and publish run clean-assets/download-assets/dotnet clean

Test plan

Open HelpMenu → "Who is using cv4pve-admin?" → GitHub issue opens pre-filled with cluster info
Open HelpMenu → "Report a Bug" → GitHub issue opens with environment pre-filled
With multiple clusters configured, verify --- separator appears between each cluster block
Run admin.ps1 -Command publish and verify it runs clean-assets + download-assets before publish
Verify HelpMenu opens quickly (no delay from API calls at init)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Corsinvest/cv4pve-admin/pull/183 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/20/2026 **Status:** ✅ Merged **Merged:** 2/20/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `main` ← **Head:** `feat/help-menu-who-is-using` --- ### 📝 Commits (1) - [`9cc81ac`](https://github.com/Corsinvest/cv4pve-admin/commit/9cc81ac23be13cac390dfc621bf13a265b028b81) feat: add "Who is using" to HelpMenu with cluster info pre-fill ### 📊 Changes **7 files changed** (+107 additions, -157 deletions) <details> <summary>View changed files</summary> 📝 `admin.ps1` (+10 -5) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/HelpMenu.razor` (+4 -3) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Components/Layout/HelpMenu.razor.cs` (+41 -25) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Helpers/ApplicationHelper.cs` (+10 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Core/Helpers/PveAdminHelper.cs` (+39 -1) 📝 `src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/Support.razor` (+3 -36) ➖ `src/Corsinvest.ProxmoxVE.Admin.Module.System/Components/Support.razor.cs` (+0 -86) </details> ### 📄 Description ## Summary - Add **"Who is using cv4pve-admin?"** entry in HelpMenu that opens a GitHub issue pre-filled with cluster data (nodes, VMs, CPU, RAM, storage) — closes #11 - Move `GenerateWhoUsingAsync` from `Support.razor.cs` to `PveAdminHelper` (shared helper) - Refactor HelpMenu to generate all URLs **lazily on-click** (avoid slow API calls at menu init time) - Add separator `---` between multiple clusters in the who-is-using body - Fix `admin.ps1 publish`: extract `Invoke-DockerPrepare` so both `build docker` and `publish` run clean-assets/download-assets/dotnet clean ## Test plan - [ ] Open HelpMenu → "Who is using cv4pve-admin?" → GitHub issue opens pre-filled with cluster info - [ ] Open HelpMenu → "Report a Bug" → GitHub issue opens with environment pre-filled - [ ] With multiple clusters configured, verify `---` separator appears between each cluster block - [ ] Run `admin.ps1 -Command publish` and verify it runs clean-assets + download-assets before publish - [ ] Verify HelpMenu opens quickly (no delay from API calls at init) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>