[PR #219] refactor: extract GetResourceUsage extensions, GetFrom helper and ClusterDetailDialog #302

New issue

Closed

opened 2026-03-07 19:57:30 +03:00 by kerem · 0 comments

kerem commented

2026-03-07 19:57:30 +03:00

Owner

Original Pull Request: https://github.com/Corsinvest/cv4pve-admin/pull/219

State: closed
Merged: Yes

Summary

Extract ResourceUsage.Get/GetSnapshots static methods into ClusterResourceExtensions.GetResourceUsage() and ClusterClientExtensions.GetResourceUsage() extension methods
Add adminService.GetFrom(ClusterNames) helper to replace repeated Where()/Any() pattern across all widgets
Simplify widgets: Gauge, GaugeStacked, Grid, NodesStatus, VmsLocked, Size (AutoSnap), Status (Updater), ResourcesEx, Summary
Extract ClusterDetailDialog component from Maps Render.razor inline code

Test plan

Verify Gauge/GaugeStacked/Grid widgets display correct CPU/Memory/Storage usage
Verify snapshot usage shown correctly where enabled
Verify Maps widget marker click opens ClusterDetailDialog
Verify NodesStatus and VmsLocked widgets work correctly

**Original Pull Request:** https://github.com/Corsinvest/cv4pve-admin/pull/219 **State:** closed **Merged:** Yes --- ## Summary - Extract `ResourceUsage.Get/GetSnapshots` static methods into `ClusterResourceExtensions.GetResourceUsage()` and `ClusterClientExtensions.GetResourceUsage()` extension methods - Add `adminService.GetFrom(ClusterNames)` helper to replace repeated `Where()/Any()` pattern across all widgets - Simplify widgets: `Gauge`, `GaugeStacked`, `Grid`, `NodesStatus`, `VmsLocked`, `Size` (AutoSnap), `Status` (Updater), `ResourcesEx`, `Summary` - Extract `ClusterDetailDialog` component from Maps `Render.razor` inline code ## Test plan - [ ] Verify Gauge/GaugeStacked/Grid widgets display correct CPU/Memory/Storage usage - [ ] Verify snapshot usage shown correctly where enabled - [ ] Verify Maps widget marker click opens ClusterDetailDialog - [ ] Verify NodesStatus and VmsLocked widgets work correctly