Skip to content

Conversation

aclark4life
Copy link
Collaborator

This PR addresses two issues with the documentation I've noticed:

  • Move known-issues to the top level
    • In addition to linking from the index, move to the top level for maximum discoverability
    • Remove from topics section due to it's cross-topic relevance.
  • Add topics section to bullet point about finding specific topics
    • Prior to this PR the topics section was only navigable from the sidebar menu and from the index under the model reference

Copy link
Collaborator

@timgraham timgraham left a comment

Choose a reason for hiding this comment

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

I'm unconvinced that we should set a precedent of including top-level documents outside of the howto/ref/topic hierarchy. Whether it's a "topic guide' could be debated. It might be a better fit in reference material.

p.s. Please don't use such vague commit messages and PR titles as "Doc fixes".

@@ -74,5 +75,6 @@ Miscellaneous
ref/index
howto/index
faq
known-issues
Copy link
Collaborator

Choose a reason for hiding this comment

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

You didn't follow the instructions above: .. Keep this toctree in sync with contents.rst.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

? You mean the ordering?

Comment on lines 26 to +28
- Looking for specific information? Try the :ref:`genindex`, :ref:`modindex`,
or the detailed :doc:`table of contents <contents>`.
:doc:`topics section <topics/index>` or detailed :doc:`table of contents
<contents>`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nor is the reference material mentioned. This text was ripped straight from Django's documentation. I don't think we need to mention every section here.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We don't need to mention every section and at first I had it in its own bullet but decided to include "topics" in a bullet about topics. 😄

Also I'm only calling out personal annoyances, not necessarily looking to symmetrize the entire docs.

Lastly, copying from Django is fine but only gets us so far.

@aclark4life aclark4life changed the title Doc fixes Move known-issues to the top level Aug 21, 2025
@aclark4life aclark4life changed the title Move known-issues to the top level Move known-issues from topics to refs Aug 21, 2025
@aclark4life

This comment was marked as resolved.

- Move known-issues to the top level
- Add topics section to bullet point about finding specific topics
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.

2 participants