[PR #231] fix: expand resource rows after data load instead of OnAfterRenderAsync #312

New issue

Closed

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

kerem commented

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

Owner

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

State: closed
Merged: Yes

Summary

Add OnDataLoaded EventCallback to ResourcesEx, fired after each data refresh
Nodes and Vms: replace OnAfterRenderAsync(firstRender) with OnDataLoaded handler to expand rows matching query string (?node= / ?vmid=)
Add _expanded guard to prevent double expansion on subsequent auto-refresh cycles

Test plan

Navigating to Resources > Nodes with ?node=pve1 auto-expands the matching node row
Navigating to Resources > VMs with ?vmid=100 auto-expands the matching VM row
No double expansion on periodic data refresh

**Original Pull Request:** https://github.com/Corsinvest/cv4pve-admin/pull/231 **State:** closed **Merged:** Yes --- ## Summary - Add `OnDataLoaded` `EventCallback` to `ResourcesEx`, fired after each data refresh - `Nodes` and `Vms`: replace `OnAfterRenderAsync(firstRender)` with `OnDataLoaded` handler to expand rows matching query string (`?node=` / `?vmid=`) - Add `_expanded` guard to prevent double expansion on subsequent auto-refresh cycles ## Test plan - [ ] Navigating to Resources > Nodes with `?node=pve1` auto-expands the matching node row - [ ] Navigating to Resources > VMs with `?vmid=100` auto-expands the matching VM row - [ ] No double expansion on periodic data refresh