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
3636 Reorganize Tahoe-LAFS manual's main page #1030
base: master
Are you sure you want to change the base?
Conversation
Links under "Indices and tables" currently do not do much: - The index is currently empty - Module index 404's on readthedocs.io - Search page is empty
It distributes your data across multiple servers. | ||
Even if some of the servers fail or are taken over by an attacker, | ||
the entire file store continues to function correctly, | ||
preserving your privacy and security. | ||
|
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.
I don't think it's necessary to provide a blurb on the Index page.
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.
I would remove the blurb if there's some reasoning on why it is unnecessary. :-)
As to why it is probably necessary: I'd imagine that a casual reader (such as someone visiting https://tahoe-lafs.readthedocs.io for the first time) would want to know, in a glance, just what it is that they are looking at.
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.
I think the Index page can just give people insight into where they can find information. If it's a casual/first-time reader, they most certainly will visit the "Welcome to Tahoe-LAFS" page. And they are bound to read the same/similar intro paragraphs, which IMO, is undesirable.
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.
I think the blurb will provide motivation to read further. Other paragraphs in "Welcome to Tahoe-LAFS" page doesn't seem to be repeating the same thing.
|
||
.. toctree:: | ||
:maxdepth: 2 | ||
:maxdepth: 1 | ||
:caption: Introduction | ||
|
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.
I would recommend renaming this to "Getting Started with Tahoe-LAFS" and include installation, and basic configuration articles". The "Using Tahoe-LAFS" section can cater to parallel stuff like some of the stuff you have mentioned (anonymizing network,CLI commands, etc).
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.
I'm not sure about this. Introduction is, well, an introduction, and the rest (installing, building, configuring, running, other usage notes) should be under "Using Tahoe-LAFS".
Please take a look at this build of the docs. Would you still re-arrange it that way?
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.
Yes, thanks for the new build. I would definitely re-arrange it that way. I am not a big fan of having an Introduction section. Feels very retro to me, like we're documenting for print media. A lot of cloud tech companies are still bound to this ideology. See "The grand theory of Documentation". I am not a big fan of this and believe this does not work with writing for the internet. Instead, we can cut the clutter and just have two types of articles "About Tech1/Feature1". How to do "Tech1/Feature1".
Sorry about the tangent, but that is the reason I don't want to call it Introduction is because for people to read introductions they have to be motivated/excited about what they're reading about. 95% of the time the user is at help because they couldn't make stuff work or just want to learn how to do specific things. And that's the reason I want all titles to (somewhat) be verbs. i.e.,
- About Tahoe-LAFS > Welcome to Tahoe-LAFS
- Getting Started > Introduction
- Installing Tahoe-LAFS > Installation Guide
- Running Tahoe-LAFS > How to Run Tahoe-LAFS
Of course, there are going to be some theoretical articles, but that's why we will have "About ..." articles; where we will actively drive the user to a "How-to" article.
About Nodes will (potentially) redirect users to:
- Configuring a Tahoe-LAFS node
- Running Tahoe-LAFS
I hope that makes sense.
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.
@YashNRam13 Made a change in f39d765. The other changes you describe will have to be separate pull requests, right?
docs/index.rst
Outdated
.. toctree:: | ||
:maxdepth: 1 | ||
:caption: Using Tahoe-LAFS | ||
|
||
INSTALL |
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.
I've removed this article and another one (OS-X packaging) in my PR. I've also made some changes to Index file (brought all installation guides at the top). I am not sure how we resolve conflicts which will arise.
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.
I have updated the PR.
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.
Awesome!
docs/index.rst
Outdated
servers | ||
.. toctree:: | ||
:maxdepth: 1 | ||
:caption: Technical Notes |
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.
I would rename this to "About Tahoe-LAFS" and if there are many parallel articles with varying complexity, then maybe "About Tahoe-LAFS in depth".
docs/index.rst
Outdated
developer-guide | ||
ticket-triage | ||
release-checklist | ||
desert-island |
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.
Wouldn't desert-island be a part of installation?
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.
That is actually geared towards Tahoe-LAFS hackers, according to the document itself:
Here’s the story: you leave for the airport in an hour, you know you want to do some Tahoe hacking on the flight. What can you grab right now that will let you install the necessary dependencies later, when you are offline?
I don't think it should be part of installation. In the current commit (d91638d) after merging master, it is under installation, but I'll move it back under developer notes if you agree about that choice.
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.
I think we can keep it near installation. We need to rework it along the lines of "Installing Tahoe-LAFS completely offline". But until we get to revamping that, we can keep it under developer notes.
@@ -5,66 +5,85 @@ Tahoe-LAFS | |||
.. Please view a nicely formatted version of this documentation at |
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.
I really like this organization of the index page. Everything is more searchable and structured.
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.
Thanks. :-)
I would like to see my top-level review comment addressed:
It doesn't matter to me very much what those principles are but I don't see how we can collaborate on the documentation without a shared understanding of them. |
@exarkun I agree with you; the documenting/organization principles for this index need to be recorded. However, I do not think that it needs to be realized by adding comments in the index itself. I'll be building off @sajith 's beautiful index here in this PR as I make changes to the documentation. We can then later work on a style guide which accommodates for this index's organization principles.
The PR description does have some of the principles, but they are highly subject to change as I find out more and more about Sphinx's and Tahoe's capabilities. As a stop-gap solution until we come up with our very own documentation style-guide; we can follow the WriteTheDocs Style Guidelines. This is not tailored for us but accommodates for everything. I've created a new trac ticket for the style guide. @exarkun if this is satisfactory, I'd love for this PR to be merged so that I can use the new index as a base for modifications. |
@YashNRam13 @exarkun @May-Lee Apologies for abandoning this PR for such a long time. Fault is all mine: I have been away, then busy with other things, then away again, and then busy again. I have added some notes to Obviously what I imagine to be true is up for debate. :-) |
Ticket is https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3636.
I humbly submit that the manual page in this version is hard to navigate, and this version is easier.
Changes:
sphinx-autogen
when generating the new Sphinx doc project.