[PR #442] [MERGED] New script: listmonk LXC #2534

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

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/442
Author: @bvdberg01
Created: 11/22/2024
Status: Merged
Merged: 11/24/2024
Merged by: @havardthom

Base: mainHead: main


📝 Commits (4)

📊 Changes

3 files changed (+209 additions, -0 deletions)

View changed files

ct/listmonk.sh (+96 -0)
install/listmonk-install.sh (+79 -0)
json/listmonk.json (+34 -0)

📄 Description

Note

We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged.

Description

Adding script for listmonk as requested in discussion https://github.com/community-scripts/ProxmoxVE/discussions/427.

I tested installation on my Proxmox installation but cannot test all functionality because I don't use this app. The update script is also tested by installing an older version from listmonk on install (changing line 40 in install/listmonk-install.sh to RELEASE="3.0.0") and then running the update script.

Type of change

Please check the relevant option(s):

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts.)

Prerequisites

The following efforts must be made for the PR to be considered. Please check when completed:

  • Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions)
  • Testing performed (I have tested my changes, ensuring everything works as expected)
  • Documentation updated (I have updated any relevant documentation)

Additional Information (optional)

Installed listmonk LXC:
image

Running update script:
image

Running update script on latest version:
image

Running listmonk:
image


🔄 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/442 **Author:** [@bvdberg01](https://github.com/bvdberg01) **Created:** 11/22/2024 **Status:** ✅ Merged **Merged:** 11/24/2024 **Merged by:** [@havardthom](https://github.com/havardthom) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (4) - [`cb147cd`](https://github.com/community-scripts/ProxmoxVE/commit/cb147cd5a7346724fe1e34e7f8eb0a275311ffcc) New script: listmonk LXC - [`6133bdc`](https://github.com/community-scripts/ProxmoxVE/commit/6133bdc42cbbbda38f400825ef7f9f2a47f4ac96) Fixed issues after PR review - [`80173bf`](https://github.com/community-scripts/ProxmoxVE/commit/80173bf360b81286d59c7c13c186563f5e3f8e12) Fixed issues after PR review - [`8a56f2e`](https://github.com/community-scripts/ProxmoxVE/commit/8a56f2e0c1f498e6541df1b6fedfdf9c9ceb9543) add network.target ### 📊 Changes **3 files changed** (+209 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `ct/listmonk.sh` (+96 -0) ➕ `install/listmonk-install.sh` (+79 -0) ➕ `json/listmonk.json` (+34 -0) </details> ### 📄 Description > [!NOTE] > We are meticulous when it comes to merging code into the main branch, so please understand that we may reject pull requests that do not meet the project's standards. It's never personal. Also, game-related scripts have a lower chance of being merged. ## Description Adding script for [listmonk](https://listmonk.app/) as requested in discussion https://github.com/community-scripts/ProxmoxVE/discussions/427. I tested installation on my Proxmox installation but cannot test all functionality because I don't use this app. The update script is also tested by installing an older version from listmonk on install (changing line 40 in install/listmonk-install.sh to RELEASE="3.0.0") and then running the update script. ## Type of change Please check the relevant option(s): - [ ] Bug fix (non-breaking change that resolves an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (a fix or feature that would cause existing functionality to change unexpectedly) - [X] New script (a fully functional and thoroughly tested script or set of scripts.) ## Prerequisites The following efforts must be made for the PR to be considered. Please check when completed: - [X] Self-review performed (I have reviewed my code, ensuring it follows established patterns and conventions) - [X] Testing performed (I have tested my changes, ensuring everything works as expected) - [X] Documentation updated (I have updated any relevant documentation) ## Additional Information (optional) **Installed listmonk LXC:** ![image](https://github.com/user-attachments/assets/d255b488-128a-4a9e-985c-d06bb3073f85) **Running update script:** ![image](https://github.com/user-attachments/assets/7cb9296f-7137-4348-9840-497c35ea114b) **Running update script on latest version:** ![image](https://github.com/user-attachments/assets/93f98e10-2d80-4ba6-8699-9375e9b0a7fe) **Running listmonk:** ![image](https://github.com/user-attachments/assets/d32d25d6-36bf-4ed8-9fac-8be69c5e9955) --- <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 05:30:31 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#2534