[PR #2409] [MERGED] Update MagicMirror install and update scripts #3652

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

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/2409
Author: @tremor021
Created: 2/14/2025
Status: Merged
Merged: 2/15/2025
Merged by: @michelroegl-brunner

Base: mainHead: magicmirror


📝 Commits (3)

📊 Changes

2 files changed (+62 additions, -32 deletions)

View changed files

📝 ct/magicmirror.sh (+39 -12)
📝 install/magicmirror-install.sh (+23 -20)

📄 Description

✍️ Description

Old scripts method of installilng and updating don't work any more, since applications update and install methods have changed. Updated the script to reflect those changes

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/2409 **Author:** [@tremor021](https://github.com/tremor021) **Created:** 2/14/2025 **Status:** ✅ Merged **Merged:** 2/15/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `magicmirror` --- ### 📝 Commits (3) - [`a9a6122`](https://github.com/community-scripts/ProxmoxVE/commit/a9a61221f4f15938517f5f361e4385c5362a47e2) Update MagicMirror install and update scripts - [`6e5068a`](https://github.com/community-scripts/ProxmoxVE/commit/6e5068a55549ed87205e3cb0a23876d3ab6b18f3) touch version file - [`a46de3f`](https://github.com/community-scripts/ProxmoxVE/commit/a46de3f4db8291348deb07c14ba585558ba6048a) fixes ### 📊 Changes **2 files changed** (+62 additions, -32 deletions) <details> <summary>View changed files</summary> 📝 `ct/magicmirror.sh` (+39 -12) 📝 `install/magicmirror-install.sh` (+23 -20) </details> ### 📄 Description ## ✍️ Description Old scripts method of installilng and updating don't work any more, since applications update and install methods have changed. Updated the script to reflect those 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:00 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3652