[PR #353] [MERGED] Add VM status check and UI improvements #380

New issue

Closed

opened 2026-02-26 12:41:11 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 12:41:11 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE-Local/pull/353
Author: @michelroegl-brunner
Created: 11/28/2025
Status: ✅ Merged
Merged: 11/28/2025
Merged by: @michelroegl-brunner

Base: main ← Head: fix/vm_detection

📝 Commits (1)

5274737 Add VM status check and UI improvements

📊 Changes

3 files changed (+545 additions, -225 deletions)

View changed files

📝 src/app/_components/InstalledScriptsTab.tsx (+36 -7)
📝 src/app/_components/ScriptInstallationCard.tsx (+3 -2)
📝 src/server/api/routers/installedScripts.ts (+506 -216)

📄 Description

This PR adds support for VM status checking and improves the UI to differentiate between LXC containers and VMs.

Changes

Add VM status checking using qm status command
Hide update button for VMs (only show for LXC containers)
Hide shell button for VMs (only show for LXC containers)
Hide LXC Settings option for VMs
Display VM/LXC indicator badges in table before script names
Update statistics cards to differentiate between LXC and VMs
Update container control to support both pct (LXC) and qm (VM) commands
Improve status parsing to handle both container types

Testing

Verified VM status checking works with qm status
Verified LXC status checking still works with pct status
Verified UI correctly hides/shows buttons based on container type
Verified statistics cards show correct counts for LXC and VMs separately

_{🔄 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/community-scripts/ProxmoxVE-Local/pull/353 **Author:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Created:** 11/28/2025 **Status:** ✅ Merged **Merged:** 11/28/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `fix/vm_detection` --- ### 📝 Commits (1) - [`5274737`](https://github.com/community-scripts/ProxmoxVE-Local/commit/5274737ab8564f8b57999306f86eff793c0c15d1) Add VM status check and UI improvements ### 📊 Changes **3 files changed** (+545 additions, -225 deletions) <details> <summary>View changed files</summary> 📝 `src/app/_components/InstalledScriptsTab.tsx` (+36 -7) 📝 `src/app/_components/ScriptInstallationCard.tsx` (+3 -2) 📝 `src/server/api/routers/installedScripts.ts` (+506 -216) </details> ### 📄 Description This PR adds support for VM status checking and improves the UI to differentiate between LXC containers and VMs. ## Changes - Add VM status checking using `qm status` command - Hide update button for VMs (only show for LXC containers) - Hide shell button for VMs (only show for LXC containers) - Hide LXC Settings option for VMs - Display VM/LXC indicator badges in table before script names - Update statistics cards to differentiate between LXC and VMs - Update container control to support both `pct` (LXC) and `qm` (VM) commands - Improve status parsing to handle both container types ## Testing - Verified VM status checking works with `qm status` - Verified LXC status checking still works with `pct status` - Verified UI correctly hides/shows buttons based on container type - Verified statistics cards show correct counts for LXC and VMs separately --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>