[PR #51] [MERGED] feat: Add Downloaded Scripts tab #143

Open
opened 2025-11-20 04:13:02 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

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

Base: mainHead: feat/downloaded_scripts_tab


📝 Commits (2)

  • 5f14810 feat: Add Downloaded Scripts tab
  • 2563b21 fix: Resolve ESLint errors in DownloadedScriptsTab

📊 Changes

6 files changed (+430 additions, -255 deletions)

View changed files

scripts/ct/2fauth.sh (+0 -82)
scripts/ct/debian.sh (+0 -43)
scripts/install/2fauth-install.sh (+0 -105)
scripts/install/debian-install.sh (+0 -24)
src/app/_components/DownloadedScriptsTab.tsx (+413 -0)
📝 src/app/page.tsx (+17 -1)

📄 Description

Summary

This PR adds a new 'Downloaded Scripts' tab between the 'Available Scripts' and 'Installed Scripts' tabs to provide better organization and visibility of downloaded scripts.

Changes

  • New DownloadedScriptsTab component: Shows only scripts that have been downloaded locally
  • Enhanced navigation: Added new tab with 💾 icon between Available and Installed Scripts
  • Consistent UI: Uses the same filtering, searching, and categorization features as other tabs
  • Statistics dashboard: Displays counts for total downloaded scripts, updatable scripts, and filtered results
  • Cleanup: Removed unused script files (2fauth.sh, debian.sh and their install scripts)

Features

  • Filter downloaded scripts by category, type, and search terms
  • Sort by name or creation date
  • Show updatable scripts count
  • Empty state with helpful guidance when no scripts are downloaded
  • Consistent design with existing tabs

Testing

  • New tab appears in navigation
  • Only downloaded scripts are shown
  • Filtering and search work correctly
  • Statistics display properly
  • Empty state shows when no scripts downloaded
  • Modal opens when clicking on script cards

Screenshots

The new tab provides a clean interface to view and manage downloaded scripts with the same functionality as the main scripts view but filtered to show only downloaded content.


🔄 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/51 **Author:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Created:** 10/7/2025 **Status:** ✅ Merged **Merged:** 10/7/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `feat/downloaded_scripts_tab` --- ### 📝 Commits (2) - [`5f14810`](https://github.com/community-scripts/ProxmoxVE-Local/commit/5f148109fb6f57696250f0db1590307725997e41) feat: Add Downloaded Scripts tab - [`2563b21`](https://github.com/community-scripts/ProxmoxVE-Local/commit/2563b21b3a0f6bbfdd6a4256bcd28e69769bd5a0) fix: Resolve ESLint errors in DownloadedScriptsTab ### 📊 Changes **6 files changed** (+430 additions, -255 deletions) <details> <summary>View changed files</summary> ➖ `scripts/ct/2fauth.sh` (+0 -82) ➖ `scripts/ct/debian.sh` (+0 -43) ➖ `scripts/install/2fauth-install.sh` (+0 -105) ➖ `scripts/install/debian-install.sh` (+0 -24) ➕ `src/app/_components/DownloadedScriptsTab.tsx` (+413 -0) 📝 `src/app/page.tsx` (+17 -1) </details> ### 📄 Description ## Summary This PR adds a new 'Downloaded Scripts' tab between the 'Available Scripts' and 'Installed Scripts' tabs to provide better organization and visibility of downloaded scripts. ## Changes - **New DownloadedScriptsTab component**: Shows only scripts that have been downloaded locally - **Enhanced navigation**: Added new tab with 💾 icon between Available and Installed Scripts - **Consistent UI**: Uses the same filtering, searching, and categorization features as other tabs - **Statistics dashboard**: Displays counts for total downloaded scripts, updatable scripts, and filtered results - **Cleanup**: Removed unused script files (2fauth.sh, debian.sh and their install scripts) ## Features - Filter downloaded scripts by category, type, and search terms - Sort by name or creation date - Show updatable scripts count - Empty state with helpful guidance when no scripts are downloaded - Consistent design with existing tabs ## Testing - [x] New tab appears in navigation - [x] Only downloaded scripts are shown - [x] Filtering and search work correctly - [x] Statistics display properly - [x] Empty state shows when no scripts downloaded - [x] Modal opens when clicking on script cards ## Screenshots The new tab provides a clean interface to view and manage downloaded scripts with the same functionality as the main scripts view but filtered to show only downloaded content. --- <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:02 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE-Local#143