[PR #2468] [MERGED] Fix: MagicMirror force Node version and fix backups #3679

Closed
opened 2025-11-20 06:06:08 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

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

Base: mainHead: magicmirror


📝 Commits (1)

  • e987b7d Ensure Node version at least v18.20.3 and make fresh backup folder each time

📊 Changes

1 file changed (+3 additions, -0 deletions)

View changed files

📝 ct/magicmirror.sh (+3 -0)

📄 Description

✍️ Description

Link: #2370

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:

  • [] 🐞 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.

📋 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/2468 **Author:** [@tremor021](https://github.com/tremor021) **Created:** 2/18/2025 **Status:** ✅ Merged **Merged:** 2/18/2025 **Merged by:** [@tremor021](https://github.com/tremor021) **Base:** `main` ← **Head:** `magicmirror` --- ### 📝 Commits (1) - [`e987b7d`](https://github.com/community-scripts/ProxmoxVE/commit/e987b7d17af7ce7b8297f4eef49cad96d5d0af1d) Ensure Node version at least v18.20.3 and make fresh backup folder each time ### 📊 Changes **1 file changed** (+3 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `ct/magicmirror.sh` (+3 -0) </details> ### 📄 Description ## ✍️ Description <!-- Provide a clear and concise description of your changes. --> ## 🔗 Related PR / Discussion / Issue Link: #2370 ## ✅ 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: - [✅] 🐞 **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. ## 📋 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:08 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3679