fix(proxmox): resolve 11 dashboard UI and API issues #127

Merged
sarman merged 6 commits from fix/proxmox-ui-and-api-issues into beta 2026-06-21 15:22:12 +00:00

6 Commits

Author SHA1 Message Date
Shaun Arman
f11f287200 fix(proxmox): resolve remaining PR review findings
All checks were successful
Test / frontend-tests (pull_request) Successful in 1m39s
Test / frontend-typecheck (pull_request) Successful in 1m47s
PR Review Automation / review (pull_request) Successful in 5m57s
Test / rust-fmt-check (pull_request) Successful in 12m41s
Test / rust-clippy (pull_request) Successful in 13m36s
Test / rust-tests (pull_request) Successful in 15m52s
- Remove non-existent get_current_proxmox_cluster command call
  Use list_proxmox_clusters and select first cluster directly
  Fixes BLOCKER: Missing Tauri command error

- Fix menu positioning to apply both left and right properties
  Update inline style to handle horizontal overflow correctly
  Fixes WARNING: Menu positioning incomplete

Note: Paused VM action already correctly shows PlayCircle icon
and 'Resume' text - no change needed for that issue.

All TypeScript checks pass. All Rust formatting checks pass.
2026-06-21 10:03:14 -05:00
fb301e889b Update .gitea/workflows/pr-review.yml
All checks were successful
Test / frontend-tests (pull_request) Successful in 1m40s
Test / frontend-typecheck (pull_request) Successful in 1m50s
PR Review Automation / review (pull_request) Successful in 4m28s
Test / rust-fmt-check (pull_request) Successful in 12m56s
Test / rust-clippy (pull_request) Successful in 14m22s
Test / rust-tests (pull_request) Successful in 16m43s
2026-06-21 14:42:13 +00:00
f9c7538405 Update .gitea/workflows/pr-review.yml
Some checks failed
PR Review Automation / review (pull_request) Has been cancelled
Test / rust-tests (pull_request) Has been cancelled
Test / frontend-typecheck (pull_request) Has been cancelled
Test / frontend-tests (pull_request) Has been cancelled
Test / rust-clippy (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
2026-06-21 14:41:30 +00:00
Shaun Arman
65ed2979e6 style: fix Rust formatting in proxmox commands
Some checks failed
PR Review Automation / review (pull_request) Has been cancelled
Test / frontend-tests (pull_request) Has been cancelled
Test / rust-clippy (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Test / rust-tests (pull_request) Has been cancelled
Test / frontend-typecheck (pull_request) Has been cancelled
2026-06-21 09:40:21 -05:00
Shaun Arman
118f817a18 fix(proxmox): address 11 dashboard issues and add missing VM action commands
Some checks failed
Test / frontend-tests (pull_request) Successful in 1m45s
Test / frontend-typecheck (pull_request) Successful in 1m54s
Test / rust-tests (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Test / rust-clippy (pull_request) Has been cancelled
PR Review Automation / review (pull_request) Successful in 6m10s
Backend:
- Add resume_proxmox_vm, suspend_proxmox_vm, clone_vm, delete_vm Tauri commands
- Implement proxmox/vm.rs functions for resume, suspend, clone, delete operations
- Register all new commands in lib.rs

Frontend:
- Fix VMList.tsx: Import confirm from @tauri-apps/plugin-dialog
- Fix VMList.tsx: Replace prompt() with window.prompt() for user input
- Fix VMList.tsx: Correct paused VM action to resume instead of suspend
- Fix VMList.tsx: Implement proper menu positioning with horizontal overflow detection
- Fix StorageList.tsx: Add error handling to formatBytes for negative/non-numeric input
- Fix VMsPage.tsx: Remove redundant handler stubs, let VMList handle actions

All changes pass TypeScript type checking, Rust clippy, and frontend tests (386 tests passing).
2026-06-21 09:38:10 -05:00
Shaun Arman
9d5390df5b fix(proxmox): resolve 11 dashboard UI and API issues
Some checks failed
Test / frontend-tests (pull_request) Successful in 1m42s
Test / frontend-typecheck (pull_request) Successful in 1m52s
PR Review Automation / review (pull_request) Successful in 4m50s
Test / rust-fmt-check (pull_request) Failing after 12m23s
Test / rust-clippy (pull_request) Successful in 13m39s
Test / rust-tests (pull_request) Successful in 15m8s
- Action menu: fix click-outside closing, positioning, opacity, and functionality
- VM metrics: fix CPU %, memory/disk bars with formatBytes helper, uptime formatting
- list_cluster_tasks: remove invalid 'limit' query parameter causing 400 error
- list_views/list_certificates: handle 501 Not Implemented gracefully
- list_proxmox_datastores: fetch per-node storage via /nodes/{node}/storage
- list_proxmox_backup_jobs: use cluster-level /cluster/backup endpoint
- Tests: update integration tests to use PROXMOX_HOST env var

Fixes:
- Action menu not closing when clicking away
- CPU/memory/disk/uptime displaying raw values
- Storage not displaying data
- Backup jobs not showing details
- Tasks API returning 400 Bad Request
- Views/Certificates APIs causing errors on older Proxmox versions
2026-06-20 23:39:33 -05:00