Skip to content
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

Updated Readme with synology instructions #62

Merged
merged 2 commits into from Apr 8, 2016
Merged

Updated Readme with synology instructions #62

merged 2 commits into from Apr 8, 2016

Conversation

ghost
Copy link

@ghost ghost commented Apr 8, 2016

:)

@@ -168,3 +168,44 @@ The container exposes /config as a volume. This is the directory where the suppl
If you have transmission authentication enabled and want scripts in another container to access and
control the transmission-daemon, this can be a handy way to access the credentials.
For example, another container may pause or restrict transmission speeds while the server is streaming video.

## Make it work an a Synology NAS
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"an a" -> "on"?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, sorry my english speaking/writing is improving everyday haha

@haugene
Copy link
Owner

haugene commented Apr 8, 2016

Great stuff, thank you! I've commented on a couple of minor things. Feel free to update them if you'd like. I'll wait to see if you do, but then I'll merge it anyways 👍

```
- Save the file with [escape] + `:wq!`
- Create your docker container with `docker run --privileged -d -v /volume1/foldername/resolv.conf:/etc/resolv.conf -v /volume1/yourpath/:/data -e "OPENVPN_PROVIDER=PIA" -e "OPENVPN_CONFIG=Netherlands" -e "OPENVPN_USERNAME=XXXXX" -e "OPENVPN_PASSWORD=XXXXX" -p 9092:9091 haugene/transmission-openvpn -name TransmissionVPN`
- If you wish to access WebUI, just run the nginx container as explained in doc on ports 9091:9091
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With the run command above, and nginx -p 9091:9091 as you say, they would have to change nginx.conf to:

Could be mentioned...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, let just not mention the WebUI, it will be confusing with your explanation you made above in the Readme

@haugene haugene merged commit 6c0400f into haugene:master Apr 8, 2016
@haugene
Copy link
Owner

haugene commented Apr 8, 2016

Great! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant