Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

New script: Koillection #2031

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

bvdberg01
Copy link
Contributor

✍️ Description

Adding script for Koillection as requested in discussion #382.
I tested the script on my Proxmox installation. The update script is also tested by installing an older version of Koillection and then running the update script.

✅ Prerequisites

The following steps must be completed for the pull request to be considered:

  • Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
  • Testing performed (I have thoroughly tested my changes and verified expected functionality.)

🛠️ Type of Change

Please check the relevant options:

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

📋 Additional Information

Installed LXC:
1
Installed old version of Koillection (with test data):
2
3
Installing update:
4
After update:
5
6
7

@bvdberg01 bvdberg01 requested a review from a team as a code owner February 4, 2025 21:00
@github-actions github-actions bot added new script A change that adds a new script website A change to the website labels Feb 4, 2025
@MickLesk MickLesk merged commit 8662be2 into community-scripts:main Feb 5, 2025
4 checks passed
@bvdberg01 bvdberg01 deleted the koillection branch February 8, 2025 08:09
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
new script A change that adds a new script website A change to the website
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants