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 #36] [MERGED] Add category sidebar and filtering to scripts grid #123

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

📋 Pull Request Information

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

Base: development ← Head: feat-category-view

📝 Commits (2)

  • 06521c6 Add category sidebar and filtering to scripts grid
  • 4fd9c59 Add category metadata to scripts and improve filtering

📊 Changes

6 files changed (+818 additions, -132 deletions)

View changed files

📝 scripts/json/metadata.json (+183 -28)
➕ src/app/_components/CategorySidebar.tsx (+363 -0)
📝 src/app/_components/ScriptDetailModal.tsx (+1 -1)
📝 src/app/_components/ScriptsGrid.tsx (+200 -103)
📝 src/server/api/routers/scripts.ts (+60 -0)
📝 src/server/services/localScripts.ts (+11 -0)

📄 Description

✍️ Description

Introduces a CategorySidebar component with icon mapping and category selection. Updates metadata.json to include icons for each category. Enhances ScriptsGrid to support category-based filtering and integrates the sidebar, improving script navigation and discoverability. Also refines ScriptDetailModal layout for better modal presentation.

🔗 Related PR / Issue

Link: #20

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

Screenshot for frontend Change

image image

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • ✨ New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.

🔄 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/36 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 10/6/2025 **Status:** ✅ Merged **Merged:** 10/6/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `development` ← **Head:** `feat-category-view` --- ### 📝 Commits (2) - [`06521c6`](https://github.com/community-scripts/ProxmoxVE-Local/commit/06521c6a4985b72b8f5dec54f90e06d6784c9231) Add category sidebar and filtering to scripts grid - [`4fd9c59`](https://github.com/community-scripts/ProxmoxVE-Local/commit/4fd9c5901eb64e8f69288f22793c385b616807d1) Add category metadata to scripts and improve filtering ### 📊 Changes **6 files changed** (+818 additions, -132 deletions) <details> <summary>View changed files</summary> 📝 `scripts/json/metadata.json` (+183 -28) ➕ `src/app/_components/CategorySidebar.tsx` (+363 -0) 📝 `src/app/_components/ScriptDetailModal.tsx` (+1 -1) 📝 `src/app/_components/ScriptsGrid.tsx` (+200 -103) 📝 `src/server/api/routers/scripts.ts` (+60 -0) 📝 `src/server/services/localScripts.ts` (+11 -0) </details> ### 📄 Description <!--🛑 All Pull Requests need to made against the development branch. PRs against main will get closed. --> ## ✍️ Description Introduces a CategorySidebar component with icon mapping and category selection. Updates metadata.json to include icons for each category. Enhances ScriptsGrid to support category-based filtering and integrates the sidebar, improving script navigation and discoverability. Also refines ScriptDetailModal layout for better modal presentation. ## 🔗 Related PR / Issue Link: #20 ## ✅ Prerequisites (**X** in brackets) - [x] **Self-review completed** – Code follows project standards. - [x] **Tested thoroughly** – Changes work as expected. - [x] **No security risks** – No hardcoded secrets, unnecessary privilege escalations, or permission issues. ## Screenshot for frontend Change <img width="1625" height="539" alt="image" src="https://github.com/user-attachments/assets/828fb08a-3c91-4df2-839a-53723548652b" /> <img width="420" height="422" alt="image" src="https://github.com/user-attachments/assets/787533c2-20c0-41e4-919c-73a2b1f3e0d1" /> --- ## 🛠️ Type of Change (**X** in brackets) - [ ] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [x] ✨ **New feature** – Adds new, non-breaking functionality. - [ ] 💥 **Breaking change** – Alters existing functionality in a way that may require updates. --- <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:12:57 -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#123
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?