[PR #9948] [MERGED] refactor: homarr #8401

Closed
opened 2026-02-26 16:32:47 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/9948
Author: @CrazyWolf13
Created: 12/14/2025
Status: Merged
Merged: 12/14/2025
Merged by: @tremor021

Base: mainHead: CrazyWolf13-refactor-homarr-prebuild


📝 Commits (10+)

📊 Changes

3 files changed (+57 additions, -169 deletions)

View changed files

📝 ct/homarr.sh (+30 -115)
📝 frontend/public/json/homarr.json (+2 -2)
📝 install/homarr-install.sh (+25 -52)

📄 Description

✍️ Description

Fully refactor homarr and implement prebuild archive support, a big thank you to the homarr team for implementing a workflow to generate prebuilt archives! @Meierschlumpf @manuel-rw

Fixes #9284

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/9948 **Author:** [@CrazyWolf13](https://github.com/CrazyWolf13) **Created:** 12/14/2025 **Status:** ✅ Merged **Merged:** 12/14/2025 **Merged by:** [@tremor021](https://github.com/tremor021) **Base:** `main` ← **Head:** `CrazyWolf13-refactor-homarr-prebuild` --- ### 📝 Commits (10+) - [`3a0b811`](https://github.com/community-scripts/ProxmoxVE/commit/3a0b81108db8c49d67884012545af89ef52ab8de) refactor: homarr - [`831b39f`](https://github.com/community-scripts/ProxmoxVE/commit/831b39fd7fcfda708f0f2944ad27a41e0a8cff96) refactor: homarr-install - [`3d8b325`](https://github.com/community-scripts/ProxmoxVE/commit/3d8b32532e048b1822d38d4ae4ba740e14050ce6) homarr: reduce resources - [`d315a18`](https://github.com/community-scripts/ProxmoxVE/commit/d315a1892601147abd273a395bae9e4afe4a6241) stylistics - [`cd3c0fe`](https://github.com/community-scripts/ProxmoxVE/commit/cd3c0fe76c373df6ac2356aec1fb01f634429823) stylistics - [`2ec0fad`](https://github.com/community-scripts/ProxmoxVE/commit/2ec0fadfba1f856dcb503c603877b0b54f725a32) Update author information in homarr.sh - [`09e0e06`](https://github.com/community-scripts/ProxmoxVE/commit/09e0e06cca53f222a5b68cf03982cbeb85e18df4) Update homarr-install.sh - [`7a81165`](https://github.com/community-scripts/ProxmoxVE/commit/7a8116591b1377cc54df919b746cbe0124f6877e) add comment - [`4b7dff7`](https://github.com/community-scripts/ProxmoxVE/commit/4b7dff79d12047b3c7e3aa697d488add0b2868a9) add: comment - [`d1b5994`](https://github.com/community-scripts/ProxmoxVE/commit/d1b5994c693fac335d538f57f395558fcf64450f) Update redis directory permissions to 744 ### 📊 Changes **3 files changed** (+57 additions, -169 deletions) <details> <summary>View changed files</summary> 📝 `ct/homarr.sh` (+30 -115) 📝 `frontend/public/json/homarr.json` (+2 -2) 📝 `install/homarr-install.sh` (+25 -52) </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 Fully refactor homarr and implement prebuild archive support, a big thank you to the homarr team for implementing a workflow to generate prebuilt archives! @Meierschlumpf @manuel-rw ## 🔗 Related Issue Fixes #9284 ## ✅ 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) - [x] 🐞 **Bug fix** – Resolves an issue without breaking functionality. - [x] ✨ **New feature** – Adds new, non-breaking functionality. - [x] 💥 **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. - [x] 🔧 **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 16:32:47 +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#8401
No description provided.