[PR #2676] [MERGED] Fix: Homarr - Manually correct db-migration wrong-folder #3810

Open
opened 2025-11-20 06:06:51 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/2676
Author: @CrazyWolf13
Created: 2/26/2025
Status: Merged
Merged: 2/26/2025
Merged by: @tremor021

Base: mainHead: fix-homarr


📝 Commits (2)

📊 Changes

2 files changed (+15 additions, -0 deletions)

View changed files

📝 ct/homarr.sh (+10 -0)
📝 install/homarr-install.sh (+5 -0)

📄 Description

✍️ Description

Out of any weird reason, homarr outputs the migration into a seperate subolder /migration, which then ends up with /opt/homarr_db/migration/migration instead of /opt/homarr_db/migration a sinple cp with recursive did not work, unless manually executed.

Will talk to homarr-devs and inform them about this bug in the migration.

Link: #

Prerequisites

Before this PR can be reviewed, the following must be completed:

  • Self-review performed – Code follows established patterns and conventions.
  • Testing performed – Changes have been thoroughly tested and verified.

🛠️ Type of Change

Select all that apply:

  • [] 🆕 New script – A fully functional and tested script or script set.
  • 🐞 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.

📋 Additional Information (optional)


🔄 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/2676 **Author:** [@CrazyWolf13](https://github.com/CrazyWolf13) **Created:** 2/26/2025 **Status:** ✅ Merged **Merged:** 2/26/2025 **Merged by:** [@tremor021](https://github.com/tremor021) **Base:** `main` ← **Head:** `fix-homarr` --- ### 📝 Commits (2) - [`6fb4638`](https://github.com/community-scripts/ProxmoxVE/commit/6fb46382f595e3835f0e57298b3b91eceb8e6953) Update homarr-install.sh - [`04826b5`](https://github.com/community-scripts/ProxmoxVE/commit/04826b550d1de1d58aa19be60fbbbf92a4684d17) Update homarr.sh ### 📊 Changes **2 files changed** (+15 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `ct/homarr.sh` (+10 -0) 📝 `install/homarr-install.sh` (+5 -0) </details> ### 📄 Description ## ✍️ Description <!-- Provide a clear and concise description of your changes. --> Out of any weird reason, homarr outputs the migration into a seperate subolder /migration, which then ends up with /opt/homarr_db/migration/migration instead of /opt/homarr_db/migration a sinple cp with recursive did not work, unless manually executed. Will talk to homarr-devs and inform them about this bug in the migration. ## 🔗 Related PR / Discussion / Issue Link: # ## ✅ Prerequisites Before this PR can be reviewed, the following must be completed: - [x] **Self-review performed** – Code follows established patterns and conventions. - [x] **Testing performed** – Changes have been thoroughly tested and verified. ## 🛠️ Type of Change Select all that apply: - [] 🆕 **New script** – A fully functional and tested script or script set. - [x] 🐞 **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. ## 📋 Additional Information (optional) <!-- Provide extra context, screenshots, or references if needed. --> --- <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 06:06:51 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3810