[PR #145] [MERGED] fix: delete associated scripts when removing server #213

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

📋 Pull Request Information

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

Base: mainHead: fix/remove_scritps_on_server_remove


📝 Commits (1)

  • a4e31ca fix: delete associated scripts when removing server

📊 Changes

3 files changed (+50 additions, -4 deletions)

View changed files

📝 src/app/_components/ServerList.tsx (+38 -3)
📝 src/app/api/servers/[id]/route.ts (+3 -0)
📝 src/server/database.js (+9 -1)

📄 Description

This PR implements automatic deletion of associated installed scripts when a server is removed, along with a confirmation modal that requires typing the server name to confirm deletion.


🔄 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/145 **Author:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Created:** 10/14/2025 **Status:** ✅ Merged **Merged:** 10/14/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `fix/remove_scritps_on_server_remove` --- ### 📝 Commits (1) - [`a4e31ca`](https://github.com/community-scripts/ProxmoxVE-Local/commit/a4e31cad76871e730e9a4b2935e26eaa16ae340b) fix: delete associated scripts when removing server ### 📊 Changes **3 files changed** (+50 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/app/_components/ServerList.tsx` (+38 -3) 📝 `src/app/api/servers/[id]/route.ts` (+3 -0) 📝 `src/server/database.js` (+9 -1) </details> ### 📄 Description This PR implements automatic deletion of associated installed scripts when a server is removed, along with a confirmation modal that requires typing the server name to confirm deletion. --- <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:33 -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#213