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

Clarify VPN doc #2317

Closed
tasket opened this Issue Sep 14, 2016 · 7 comments

Comments

Projects
None yet
2 participants
@tasket

tasket commented Sep 14, 2016

Clarify language to address the issues some people had: Configuring without testing, mixing NM with the scripts, basic troubleshooting.
Also make the language less openvpn-specific so it is only shown as examples.


Related issues:

#1941

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
Member

andrewdavidwong commented Sep 15, 2016

Did you mean to submit this as a PR against https://github.com/QubesOS/qubes-doc/blob/master/configuration/vpn.md ?

@andrewdavidwong andrewdavidwong added this to the Documentation/website milestone Sep 15, 2016

@tasket tasket referenced this issue in QubesOS/qubes-doc Sep 15, 2016

Merged

Update vpn.md #186

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Sep 15, 2016

Yes, sorry. Have new PR.

tasket commented Sep 15, 2016

Yes, sorry. Have new PR.

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Sep 17, 2016

Two formatting problems at https://www.qubes-os.org/doc/vpn/ ...

Step 4 has a multi-line code box that didn't render... lines run together.

The Usage section has an image that isn't displaying.

tasket commented Sep 17, 2016

Two formatting problems at https://www.qubes-os.org/doc/vpn/ ...

Step 4 has a multi-line code box that didn't render... lines run together.

The Usage section has an image that isn't displaying.

andrewdavidwong added a commit to QubesOS/qubes-doc that referenced this issue Sep 17, 2016

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
Member

andrewdavidwong commented Sep 17, 2016

@tasket: Fixed.

@andrewdavidwong

This comment has been minimized.

Show comment
Hide comment
@andrewdavidwong

andrewdavidwong Sep 19, 2016

Member

@tasket: Anything else needed for this issue?

Member

andrewdavidwong commented Sep 19, 2016

@tasket: Anything else needed for this issue?

@tasket

This comment has been minimized.

Show comment
Hide comment
@tasket

tasket Sep 19, 2016

Nope. Thanks!

tasket commented Sep 19, 2016

Nope. Thanks!

@tasket tasket closed this Sep 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment