Skip to content

unraid.xml UX changes

hofq requested to merge gammeltonne1/crafty-4:dev into master

What does this MR do

Better UX for Unraid Users

and why?

Port 8123 -> 8223 [Host]

8123 is used by Home-Assistant. Since it is very commonly used we should just use another Port.

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.

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