-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Enhance documentation readability. #2095
Conversation
b95b883
to
cb7d84e
Compare
6f20487
to
77c0c30
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What a awesome PR!!!
Just a problem with a broken link.
docs/user-guide/examples.md
Outdated
DNS challenge needs environment variables to be executed. This variables have to be set on the machine/container which host Traefik. | ||
DNS challenge needs environment variables to be executed. | ||
This variables have to be set on the machine/container which host Traefik. | ||
|
||
These variables has described [in this section](toml/#acme-lets-encrypt-configuration). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link in this section
does not see to still working, replace it by configuration/acme/#dnsprovider
77c0c30
to
40b4297
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 📚 👏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
/ping
( Separate ping from dashboard? #2094)Fixes #2097