search

linuxserver / docker-snipe-it

Alpine/Nginx container for the Asset Management software Snipe-IT
GNU General Public License v3.0
123 stars 26 forks source link

rework init logic update readme and reformat files #36

Closed thelamer closed 2 years ago

thelamer commented 2 years ago

closes #35 closes #30 closes #25 closes #20 closes #32

The NGINX_APP_URL param implemented does not work anymore and APP_URL is required now. This change ensures that people using just NGINX_APP_URL or APP_URL or both will still be functional with all the correct settings.

I also pulled out the custom compose stuff and allow user to set timezone.

LinuxServer-CI commented 2 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v5.4.0-pkg-cb199abf-pr-36/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v5.4.0-pkg-cb199abf-pr-36/shellcheck-result.xml

thelamer commented 2 years ago

CI is borked, I am leaving on the web stuff for now as maybe there can be a solution down the line. But the app redirects for setup require them to be set to a hostname or IP even if using a sqlite backend for just displaying the setup page.

This image does function when plugged into a DB and the APP_URL or NGINX_APP_URL is set properly.

LinuxServer-CI commented 2 years ago

I am a bot, here are the test results for this PR: https://ci-tests.linuxserver.io/lspipepr/snipe-it/v5.4.0-pkg-cb199abf-pr-36/index.html https://ci-tests.linuxserver.io/lspipepr/snipe-it/v5.4.0-pkg-cb199abf-pr-36/shellcheck-result.xml