SVI/ProxmoxVE-Local
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE-Local.git synced 2026-03-31 06:23:54 -04:00
Code Issues 56 Packages Projects Releases 7 Wiki Activity

[PR #123] [MERGED] feat: Add container running status indicators with auto-refresh #200

New Issue
Closed
opened 2025-11-20 04:13:29 -05:00 by saavagebueno · 0 comments
saavagebueno commented 2025-11-20 04:13:29 -05:00
Owner
Copy Link

📋 Pull Request Information

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

Base: main ← Head: feat/show_lxc_running_state

📝 Commits (5)

  • 762d748 feat: add container running status indicators with auto-refresh
  • dc62a08 feat: update default sorting to group by server then container ID
  • b39d85f feat: improve container status check triggering
  • f02fe1f perf: optimize container status checking by batching per server
  • 23f7501 fix: resolve all linting errors

📊 Changes

3 files changed (+340 additions, -10 deletions)

View changed files

📝 src/app/_components/InstalledScriptsTab.tsx (+133 -9)
📝 src/app/_components/ScriptInstallationCard.tsx (+24 -1)
📝 src/server/api/routers/installedScripts.ts (+183 -0)

📄 Description

🚀 Container Running Status Feature

This PR adds real-time container running status indicators to the installed scripts tab with automatic refresh every 60 seconds.

✨ Features

  • Visual Status Indicators: Green/red dots next to container IDs showing running/stopped status
  • Auto-Refresh: Status updates automatically every 60 seconds
  • Multi-Server Support: Works with both local and SSH remote containers
  • Performance Optimized: Batches status checks per server (1 call per server vs 1 per container)
  • Responsive Design: Works on both desktop table and mobile card views
  • Manual Refresh: Added refresh button for on-demand status checking

🔧 Technical Implementation

  • Backend: New tRPC endpoint with batched processing
  • Frontend: Updated UI components with status indicators and auto-refresh logic
  • Performance: Optimized from N SSH calls to 1 call per server
  • TypeScript: Full type safety with updated interfaces

📊 Performance Impact

  • Before: 10 containers = 10 SSH calls
  • After: 10 containers on 1 server = 1 SSH call (10x faster)

🎯 User Experience

  • Status indicators appear immediately when switching to installed scripts tab
  • Real-time updates every 60 seconds
  • Clear visual feedback: 🟢 Running, 🔴 Stopped, ⚪ Unknown
  • Manual refresh option for immediate updates

🧪 Testing

  • Local container status checking
  • SSH remote container status checking
  • Auto-refresh functionality
  • UI responsiveness
  • Error handling and fallbacks

Resolves container status visibility in installed scripts tab.

🔄 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/123 **Author:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Created:** 10/13/2025 **Status:** ✅ Merged **Merged:** 10/13/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `feat/show_lxc_running_state` --- ### 📝 Commits (5) - [`762d748`](https://github.com/community-scripts/ProxmoxVE-Local/commit/762d7485e8310ee044bcc3ab752b0d9cf95e49fd) feat: add container running status indicators with auto-refresh - [`dc62a08`](https://github.com/community-scripts/ProxmoxVE-Local/commit/dc62a08360b146b06d9414dcfcbded773472d10f) feat: update default sorting to group by server then container ID - [`b39d85f`](https://github.com/community-scripts/ProxmoxVE-Local/commit/b39d85f519eb03cbf11f81be102fed020ac2e74e) feat: improve container status check triggering - [`f02fe1f`](https://github.com/community-scripts/ProxmoxVE-Local/commit/f02fe1fe2a51c91bf70a5a226b2aa6976c222d4a) perf: optimize container status checking by batching per server - [`23f7501`](https://github.com/community-scripts/ProxmoxVE-Local/commit/23f75011748d10b06d9f26e5be443e3af55d9a7a) fix: resolve all linting errors ### 📊 Changes **3 files changed** (+340 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `src/app/_components/InstalledScriptsTab.tsx` (+133 -9) 📝 `src/app/_components/ScriptInstallationCard.tsx` (+24 -1) 📝 `src/server/api/routers/installedScripts.ts` (+183 -0) </details> ### 📄 Description ## 🚀 Container Running Status Feature This PR adds real-time container running status indicators to the installed scripts tab with automatic refresh every 60 seconds. ### ✨ Features - **Visual Status Indicators**: Green/red dots next to container IDs showing running/stopped status - **Auto-Refresh**: Status updates automatically every 60 seconds - **Multi-Server Support**: Works with both local and SSH remote containers - **Performance Optimized**: Batches status checks per server (1 call per server vs 1 per container) - **Responsive Design**: Works on both desktop table and mobile card views - **Manual Refresh**: Added refresh button for on-demand status checking ### 🔧 Technical Implementation - **Backend**: New tRPC endpoint with batched processing - **Frontend**: Updated UI components with status indicators and auto-refresh logic - **Performance**: Optimized from N SSH calls to 1 call per server - **TypeScript**: Full type safety with updated interfaces ### 📊 Performance Impact - **Before**: 10 containers = 10 SSH calls - **After**: 10 containers on 1 server = 1 SSH call (10x faster) ### 🎯 User Experience - Status indicators appear immediately when switching to installed scripts tab - Real-time updates every 60 seconds - Clear visual feedback: 🟢 Running, 🔴 Stopped, ⚪ Unknown - Manual refresh option for immediate updates ### 🧪 Testing - [x] Local container status checking - [x] SSH remote container status checking - [x] Auto-refresh functionality - [x] UI responsiveness - [x] Error handling and fallbacks Resolves container status visibility in installed scripts tab. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 04:13:29 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
enhancement
enhancement
feature
in progress
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
saavagebueno
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE-Local#200
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?