unraid.xml UX changes
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.
- Open Unraid Web-Terminal
- 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
- 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