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

Add flux docs assessment #89

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

celestehorgan
Copy link
Contributor

Closes #59 by adding the Flux docs assessment

cc @alisondy - Are you still a maintainer for Flux? if not, who would you like to pass this review on to?

Copy link
Collaborator

@chalin chalin left a comment

Choose a reason for hiding this comment

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

Looks good!

assessments/0005-fluxcd.md Show resolved Hide resolved
assessments/0005-fluxcd.md Show resolved Hide resolved
@celestehorgan
Copy link
Contributor Author

Upon further reflection I'm going to revisit the final conclusions section of this one tomorrow/later today - I think there's more to say and I just got a bit lazy/tired.

@dholbach
Copy link

👋 Looping in a few Flux maintainers.

cc @darkowlzz @hiddeco @kingdonb @makkes @phillebaba @relu @scottrigby @squaremo @stefanprodan

@dholbach
Copy link

dholbach commented Dec 15, 2021

Thanks so much for the review - I just found the PR today. I'll bring it up with the Flux maintainers and we'll review this together.

Please let us know if there any next steps, if we should come to one of your next meetings, or if there's a way for us to benefit some more from your experience!

This is really great! 🏆 ❤️

- Items under the **Project** top level navigation section. At a glance I couldn't figure out how these were being pulled in via (I assume) the [Community repository](https://github.com/fluxcd/community), meaning they suffer either from a lack of documentation or unclear automation.

- Documentation versioning: Flux's documentation is not versioned but the tool itself has been versioned.
- This leaves existing users of v1 who haven't upgraded in a difficult position: if they need to troubleshoot an issue, they can't access v1 documentation.
Copy link

@kingdonb kingdonb Dec 15, 2021

Choose a reason for hiding this comment

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

While this is apparently not transparent enough given you did not find it, the legacy docs which were homed at https://docs.fluxcd.io are still there, and we have preserved all existing hyperlinks (hopefully). We do not draw attention to them and in fact we've done everything we can to downplay their SEO, which also has the unfortunate side-effect that they are not searchable anymore outside of the site.

I was pretty sure we had rendered them completely un-searchable because of the difficulty of maintaining two parallel search engines for the same domain. But I just checked, and users who find https://docs.fluxcd.io or https://fluxcd.io/legacy/flux/ that it redirects to, can in fact search Flux v1 docs, and they appear to be every bit as functionally accessible as they ever were (when you're able to find them at all.)

Hope this helps! Thanks!

Choose a reason for hiding this comment

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

I had a review holding in mind a question, "how are users (that inherit a Flux v1 installation, lets say) supposed to find the legacy docs" and it looks like there's now only one straightforward way, on the front page, about 11 pages of content in (or about 80% of the way down the scroll bar.) I opened an issue so we can address this 👍 fluxcd/website#689

Copy link
Member

@nate-double-u nate-double-u left a comment

Choose a reason for hiding this comment

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

As folks are already opening Issues against this assessment, let's merge it as is. We can update as needed with feedback from the project.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flux docs assessment
5 participants