-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix compose file #18
Comments
You know, I just had the thought to instead of make it a .env value, to get the hostname from the PC itself. That might be easier. |
@gljones2001 I would gladly accept your pull request with above mentioned enhancement |
oh. I didn't have a pull request. I just manually fixed it by using the web editor in Portainer. |
Honestly it was such a simple edit I thought you could just do it through the github web editor. I didn't want to do a whole pull request just for like 2 lines of code. |
fix docker-compose file: parametrize hostname ( closes #18 )
Awesome dude. Thanks. I'll try it out. |
Hey, you might want to update the setup page in the docs. and the portainer page. |
Also, Noticed a descrepicy in the .env on the readme.
Notice the |
good catch |
I had to edit the compose file to fix an issue of Traefik not allowing me to access the webpage from another machine.
I would get a
404 Page Not Found
error uintill i did the fix below.papermerge.local
with${HOSTNAME}
indocker-compose.yml
HOSTNAME=<hostname of PC>
to .envThe text was updated successfully, but these errors were encountered: