Skip to content

New cool unraid xml update

What does this MR do

Better UX for Unraid Users

and why?

Port 8123 Notice [Host]

Reverted

Removed 8000 [Not Actively used]

Just a Redirect. For Unraid it doesn't seem to be needed and is just blocking Port 8000 for other apps.

Updated Explanation

I first didn't know if it was for Importing Servers or Data from earlier Crafty Instances, so i updated the description

Changed Web UI explanation

Since there is only 1 Web UI Port now, i updated the Description to make it easier to read

Crafty-4 -> crafty-4

Most Application Folder in Unraid are named lower case. We should do that too.

Template Rename Crafty -> Crafty-4

If we kept the Name "Crafty" it would overwrite old Templates of Crafty 3 without the Users agreement.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Open Unraid Web-Terminal
  2. Run cd /boot/config/plugins/dockerMan/templates-user && wget https://gitlab.com/gammeltonne1/crafty-4/-/raw/dev/docker/unraid.xml && mv unraid.xml crafty-4-gt.xml
  3. Select the Template in Unraid Web UI

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

  • Have you checked this doesn't interfere/conflict/duplicate someone elses work?
  • Have you fully tested your changes?
  • Have you resolved any lint issues?
  • Have you assigned a reviewer?
  • Have you applied correct labels?
Edited by hofq

Merge request reports

Loading