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

[DOCS] Complete overhaul of readthedocs #847

Merged
merged 28 commits into from
Dec 27, 2023
Merged

Conversation

Svagtlys
Copy link
Contributor

This is an ongoing project to revamp the YTDL-Sub docs. Starting point:

  • Recreate docs folder/file structure
  • Update root makefile to use new make docs command
  • Move old docs into new docs folder structure (as a base)
  • Separate out old docs into individual files to allow better navigation with the new theme

Initial plans:

  • step-by-step layman install instructions for each platform
  • pages for each pre-built preset, showing which variables can be overridden to do different things (i.e. episode_title)
  • new wiki walkthrough that uses the README config as a starting point, and gradually adds custom changes
  • pages for how to enable custom metadata agents for Kodi/Plex/jellyfin
  • page dedicated to explaining the structure of a config (it's only lightly touched on now, many folks struggle to understand the 'pattern' of presets/plugins/overrides)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
This is an ongoing project to revamp the YTDL-Sub docs.
Starting point:

- Recreate docs folder/file structure
- Update root makefile to use new make docs command
- Move old docs into new docs folder structure (as a base)
- Separate out old docs into individual files to allow better navigation with the new theme
This is an ongoing project to revamp the YTDL-Sub docs.
Starting point:

- Recreate docs folder/file structure
- Update root makefile to use new make docs command
- Move old docs into new docs folder structure (as a base)
- Separate out old docs into individual files to allow better navigation with the new theme
@Svagtlys Svagtlys mentioned this pull request Dec 19, 2023
Copy link

codecov bot commented Dec 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b8fb119) 96.99% compared to head (4fe8bed) 96.99%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #847   +/-   ##
=======================================
  Coverage   96.99%   96.99%           
=======================================
  Files         116      116           
  Lines        7115     7116    +1     
=======================================
+ Hits         6901     6902    +1     
  Misses        214      214           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Also some minor accessibility updates (alt text on intro images)
Your first configuration will look pretty simple:

.. code-block:: yaml
:linenos:
Copy link
Owner

Choose a reason for hiding this comment

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

Very nice

.. code-block:: yaml
:linenos:

configuration:
Copy link
Owner

Choose a reason for hiding this comment

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

Let's make indents 2, that has been the historical value to use in all codedocs


Now that you have installed ``ytdl-sub``, checked your skills, and gotten a bit of background on how ``ytdl-sub`` functions, read through the articles below to get started:

:doc:`Step 1: Initial Configuration <first_config>`
Copy link
Owner

Choose a reason for hiding this comment

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

I think we should have a step prior to this that explains on how to just use the subscription before jumping into config building


Line 4 begins the definition o your custom ``presets``, with line 5 being the name of your first custom ``preset``.

Lines 7 and 8 tell ``ytdl-sub`` which :doc:`/guides/getting_started/prebuilt_presets` to expand on; these ``presets`` already indicate that the downloaded files should be:
Copy link
Owner

Choose a reason for hiding this comment

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

In the docs, its noted as Example Configs when it should be Prebuilt Presets

The first two lines in this ``config.yaml`` file are the ``configuration``, and define the ``working_directory``, which is described near the bottom of :ref:`this section <guides/getting_started/index:quick overview of \`\`ytdl-sub\`\`>`


Line 4 begins the definition o your custom ``presets``, with line 5 being the name of your first custom ``preset``.
Copy link
Owner

Choose a reason for hiding this comment

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

type: o -> of


The first line in this subscription file is the ``preset``, which provides the "definitions" for the subscription as listed in :doc:`/guides/getting_started/first_config`.

The second line is the actual ``subscription``, named ``Rick Astley``, with a link to a :yt-dlp:`yt-dlp supported site <blob/master/supportedsites.md>`, in this case a YouTube channel.
Copy link
Owner

Choose a reason for hiding this comment

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

Should probably mention somewhere that __preset__ values now live in the config itself

Initial Subscription
====================

Your first subscription should look similar to the below:
Copy link
Owner

Choose a reason for hiding this comment

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

Should mention something like "Using our config built in , your first subscription..."


.. tab-set::

.. tab-item:: GUI Image
Copy link
Owner

Choose a reason for hiding this comment

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

Amazing!


:doc:`Step 4: Automating Downloads <automating_downloads>`

:doc:`Advanced Configuration <advanced_configuration>`
Copy link
Owner

Choose a reason for hiding this comment

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

This should live before Initial Configuration IMO to lay the groundwork of what a config.yaml looks like

Entry Variables
===============

.. autoclass:: ytdl_sub.entries.script.variable_definitions.VariableDefinitions()
Copy link
Owner

Choose a reason for hiding this comment

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

I need to poke around and figure out how to get each property (variable) to show on the right-hand side - this page not very helpful otherwise

},
],
"announcement": (
"Migration to <a href='https://ytdl-sub--841.org.readthedocs.build/en/841/config.html#beautifying-subscriptions'>beautiful subscriptions</a> is now live"
Copy link
Owner

Choose a reason for hiding this comment

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

Let's make this say that docs are heavily WIP

Doesn't work quite right, exploring other options for generating docs with Jesse
@Svagtlys Svagtlys marked this pull request as ready for review December 24, 2023 16:24
@Svagtlys
Copy link
Contributor Author

Ready for review to allow for parallel work

faq/index
deprecation_notices

.. note:: End goal similar to: https://picard-docs.musicbrainz.org/en/functions/list_by_type.html
Copy link
Owner

Choose a reason for hiding this comment

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

Let's remove this

Copy link
Owner

Choose a reason for hiding this comment

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

Or better yet make like a two sentence statement about how the docs are an active WIP

@jmbannon jmbannon changed the title Doc revamp [DOCS] Complete overhaul of readthedocs Dec 26, 2023
@jmbannon jmbannon merged commit 008bcf1 into jmbannon:master Dec 27, 2023
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