Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

read-the-docs follow-up #2303

Closed
8 of 11 tasks
dholbach opened this issue Jul 29, 2019 · 4 comments
Closed
8 of 11 tasks

read-the-docs follow-up #2303

dholbach opened this issue Jul 29, 2019 · 4 comments
Labels
docs Issue or PR relates to documentation

Comments

@dholbach
Copy link
Member

dholbach commented Jul 29, 2019

Following #2152 we should take a look at:

Content

  • factor in Hidde's feedback
    • 'Installing Flux' index should contain a section about installing Flux and the Helm operator at the same time, this could be on a new page or we can divide the 'get started' in two sections, one without the operator and one with.
    • I think we should rethink some of the page titles, e.g. we have a top item 'Using Flux' with a 'child page' 'Using Weave Flux'.
    • I think we should rethink some parts of the (naming of the) new structure, e.g, we have a 'introducing Weave Flux' index, but also a 'how Weave Flux Works' index. As a new user, I would expect the introduction to contain the latter, so we may need to rethink the naming, or merge parts.
    • 'Internals' is something that would scare me away as a user, listing the more important features under this index (garbage collection and manifest factorization) will probably reduce the amount of people finding this information by themselves.
  • Helm Operator docs should be in a separate dir
  • fix references from Weaveworks website

Build

  • generally document how to use this

Publish

  • pull in older releases
  • link to docs from main README.md
  • docs.fluxcd.io
@dholbach dholbach added the docs Issue or PR relates to documentation label Jul 29, 2019
@demisx
Copy link

demisx commented Jul 29, 2019

Would be also helpful if all links in the Readme were working. Currently, a few of them return 404 Not Found error from GitHub.

dholbach pushed a commit that referenced this issue Jul 30, 2019
	- not covered by linkchecker
	- dropped duplicate install/index
	- fixes part of #2303
dholbach pushed a commit that referenced this issue Jul 30, 2019
	- not covered by linkchecker
	- dropped duplicate install/index
	- fixes part of #2303
@dholbach
Copy link
Member Author

Thanks @demisx for letting us know. Proposed a fix for this in #2307.

@stefanprodan
Copy link
Member

@dholbach I think we can close this and open specific issues for dead links if there are any left.

@dholbach
Copy link
Member Author

Big +1. I think we're good for now! :-D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Issue or PR relates to documentation
Projects
None yet
Development

No branches or pull requests

3 participants