[PR #225] refactor: simplify VM/Node managers; add query param navigation #308

New issue

Closed

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

kerem commented

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

Owner

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

State: closed
Merged: Yes

Summary

Move tab visibility logic from C# code-behind to Razor @if directives in VM Manager
Remove CanConsole, CanPowerManagement, SelectedIndexChangedAsync — logic delegated to ToolBar component
Add ?node= query param support in Node Manager for direct deep-linking to a specific node
Simplify ResourcesEx and related cluster components

Test plan

Verify VM detail tabs (Console, Snapshots, Backup, etc.) render correctly
Verify Node Manager opens correctly with and without ?node= query param
Verify VM/Node toolbar actions still work

**Original Pull Request:** https://github.com/Corsinvest/cv4pve-admin/pull/225 **State:** closed **Merged:** Yes --- ## Summary - Move tab visibility logic from C# code-behind to Razor `@if` directives in VM Manager - Remove `CanConsole`, `CanPowerManagement`, `SelectedIndexChangedAsync` — logic delegated to ToolBar component - Add `?node=` query param support in Node Manager for direct deep-linking to a specific node - Simplify `ResourcesEx` and related cluster components ## Test plan - [ ] Verify VM detail tabs (Console, Snapshots, Backup, etc.) render correctly - [ ] Verify Node Manager opens correctly with and without `?node=` query param - [ ] Verify VM/Node toolbar actions still work