[PR #11563] [MERGED] feat(frontend): add JSON script import functionality #9579

Closed
opened 2026-02-26 17:30:58 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/11563
Author: @ls-root
Created: 2/5/2026
Status: Merged
Merged: 2/7/2026
Merged by: @MickLesk

Base: mainHead: feature/import-json


📝 Commits (2)

  • 38bd3b6 feat(frontend): add JSON script import functionality
  • d0622b7 Merge branch 'main' into feature/import-json

📊 Changes

2 files changed (+168 additions, -2 deletions)

View changed files

📝 frontend/src/app/json-editor/page.tsx (+167 -1)
📝 frontend/src/components/command-menu.tsx (+1 -1)

📄 Description

✍️ Description

Allow users to import scripts into the JSON editor via local file upload or by selecting from the existing script library.

  • Dropdown toggle for local and existing script import sources.
  • Script selection dialog with category and search filters.
  • Global search using the existing command menu logic.
Import dropdown menu

Note

The dropdown shouldn't be transparent. I'm sure it has to do something with my host system and hyprland
Import existing script dialog

Import existing script dialog with selected category

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.

🛠️ 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.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔄 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/pull/11563 **Author:** [@ls-root](https://github.com/ls-root) **Created:** 2/5/2026 **Status:** ✅ Merged **Merged:** 2/7/2026 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `feature/import-json` --- ### 📝 Commits (2) - [`38bd3b6`](https://github.com/community-scripts/ProxmoxVE/commit/38bd3b674f4b946349578b29695b6194b9c941ef) feat(frontend): add JSON script import functionality - [`d0622b7`](https://github.com/community-scripts/ProxmoxVE/commit/d0622b719b9ba4f799d88a668da0ca04d50d973b) Merge branch 'main' into feature/import-json ### 📊 Changes **2 files changed** (+168 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `frontend/src/app/json-editor/page.tsx` (+167 -1) 📝 `frontend/src/components/command-menu.tsx` (+1 -1) </details> ### 📄 Description <!--🛑 New scripts must be submitted to [ProxmoxVED](https://github.com/community-scripts/ProxmoxVED) for testing. PRs without prior testing will be closed. --> ## ✍️ Description Allow users to import scripts into the JSON editor via local file upload or by selecting from the existing script library. - Dropdown toggle for local and existing script import sources. - Script selection dialog with category and search filters. - Global search using the existing command menu logic. <img width="1481" height="300" alt="Import dropdown menu " src="https://github.com/user-attachments/assets/9546343a-ef22-4d14-8ae9-fb9b8560d5f3" /> > [!NOTE] > The dropdown shouldn't be transparent. I'm sure it has to do something with my host system and hyprland <img width="849" height="433" alt="Import existing script dialog" src="https://github.com/user-attachments/assets/89d893e9-eb57-4e3f-a346-c37e07f8de11" /> <img width="853" height="865" alt="Import existing script dialog with selected category" src="https://github.com/user-attachments/assets/cca401d1-d191-421d-bbc7-6a24fddab7ee" /> ## ✅ 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. --- ## 🛠️ 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. - [ ] 🆕 **New script** – A fully functional and tested script or script set. - [x] 🌍 **Website update** – Changes to website-related JSON files or metadata. - [ ] 🔧 **Refactoring / Code Cleanup** – Improves readability or maintainability without changing functionality. - [ ] 📝 **Documentation update** – Changes to `README`, `AppName.md`, `CONTRIBUTING.md`, or other docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 17:30:58 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxmoxVE#9579
No description provided.