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

Re-organize documentation #2012

Merged
merged 5 commits into from
Aug 28, 2017

Conversation

jmaitrehenry
Copy link
Contributor

@jmaitrehenry jmaitrehenry commented Aug 25, 2017

Description

  • Create a Backend section for the documentation and split toml page into different backends pages
  • Create a configuration section, put traefik.toml and acme pages here.
  • Update some list into table (API, Consul)
  • sync traefik.sample.toml and documentation.

It could be interesting to move user-guide in each associated backend, the HA/cluster into Configuration.

@ldez
Copy link
Contributor

ldez commented Aug 25, 2017

Awesome job ❤️
Could you rebase on master?

@jmaitrehenry
Copy link
Contributor Author

@ldez Done

@ldez
Copy link
Contributor

ldez commented Aug 25, 2017

@jmaitrehenry Due to the size of the PR, if necessary, do you allow us to change your PR instead of making a painful/long review?

@jmaitrehenry
Copy link
Contributor Author

jmaitrehenry commented Aug 25, 2017 via email

jmaitrehenry and others added 5 commits August 28, 2017 20:36
- user-guide review.
- add DataDog and StatD configuration.
- sync sample.toml and doc.
- split entry points doc.
- Deprecated.
@traefiker traefiker force-pushed the toml_split_in_different_pages branch from c0ee17a to c731222 Compare August 28, 2017 20:36
@traefiker traefiker merged commit ffb53c0 into traefik:master Aug 28, 2017
@jmaitrehenry jmaitrehenry deleted the toml_split_in_different_pages branch August 28, 2017 21:14
@SerialVelocity
Copy link

Hey @jmaitrehenry and @ldez,

The documentation/example about "forward auth" seems to have disappeared due to this PR. Was that intentional?

Best,
Ben

@ldez
Copy link
Contributor

ldez commented Sep 17, 2017

@SerialVelocity it's already fixed #2110

@SerialVelocity
Copy link

Nice, thank you! ❤️

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.

6 participants