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

Improve the wording in the documentation for Docker and fix title for Docker User Guide #3797

Merged
merged 2 commits into from
Aug 20, 2018
Merged

Improve the wording in the documentation for Docker and fix title for Docker User Guide #3797

merged 2 commits into from
Aug 20, 2018

Conversation

dduportal
Copy link
Contributor

What does this PR do?

This Pull Request introduce 2 changes:

  • Improving the wording in the documentation for the Docker configuration page,
    about the domain directive

  • Fixing the title of the documentation's user guide about Let's Encrypt & Docker

Motivation

  • While trying to reproduce some basic user configuration issue in the Slack support,
    I was stuck on a basic use case: "1 container behind 1 traefik". Being too quick, I directly read the Docker's Configuration page, where it was saying "docker.domain: set the domain to use". This simple rewording put an attention point for the newcomer, so they will search for the rule, and won't be stuck on this. Having checked this with @nmengin confirmed me it could be a good idea to reword (Hence the rewording I propose is not an absolute one :) )
  • While proposing ths rewording, I realized that the title of the User guide about Let's Encrypt & Docker was not matching its table of content's item.

More

  • [-] Added/updated tests
  • Added/updated documentation

Additional Notes

… page, about the 'domain' directive

Signed-off-by: Damien DUPORTAL <damien.duportal@gmail.com>
…ocker

Signed-off-by: Damien DUPORTAL <damien.duportal@gmail.com>
Copy link
Contributor

@dtomcej dtomcej left a comment

Choose a reason for hiding this comment

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

LGTM
:shipit:

Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

LGTM 📖

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

@ldez ldez added this to the 1.7 milestone Aug 20, 2018
@traefiker traefiker merged commit e46de74 into traefik:v1.7 Aug 20, 2018
@dduportal dduportal deleted the docs-hostname-rule-improvement branch February 18, 2019 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants