Skip to content

Update documentation links to full URL#1

Closed
pamelafox wants to merge 1 commit intohamelsmu:mainfrom
pamelafox:patch-1
Closed

Update documentation links to full URL#1
pamelafox wants to merge 1 commit intohamelsmu:mainfrom
pamelafox:patch-1

Conversation

@pamelafox
Copy link
Copy Markdown
Contributor

The current relative URLs are 404ing, as those HTML files dont exist at root.

Apparently the README.md file is autogenerated, so am not sure where is the right place to update them?

The current relative URLs are 404ing, as those HTML files dont exist at root.

Apparently the README.md file is autogenerated, so am not sure where is the right place to update them.
@hamelsmu
Copy link
Copy Markdown
Owner

hamelsmu commented Apr 3, 2026

Thank you! Just seeing this

@hamelsmu
Copy link
Copy Markdown
Owner

hamelsmu commented Apr 3, 2026

Thanks for flagging this, @pamelafox! You were right — the relative doc links were broken because the referenced pages (yt.html, zoom.html, writing.html) were never being generated (the notebooks weren't included in the sidebar config).

Rather than patching the URLs to absolute paths, I've addressed the root cause:

  • Created new .qmd doc pages for YouTube and X/Twitter utilities (replacing the stale notebook)
  • Updated sidebar.yml to include all doc pages
  • Fixed the links in index.ipynb (which auto-generates the README) to point to the correct pages
  • Used Quarto's {{< include >}} to pull CLI reference docs from the skill definitions, so they stay in sync automatically

Working on getting this merged your observation kicked off a proper docs overhaul. Much appreciated!

@hamelsmu hamelsmu closed this in b5d7c41 Apr 3, 2026
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