Skip to content

Conversation

@taddes
Copy link
Collaborator

@taddes taddes commented Jan 8, 2026

Description

Migrates old docs published to https://mozilla-services.readthedocs.io/en/latest/index.html. Converts from .rst to .md and used mdbook. Fixes broken graphviz diagrams and converts to mermaid. Publishes to our GitHub Pages.

I have done my best to mark outdated docs that we can consider sunsetting, or marking deprecated. Regardless, some of those are maintained here for legacy. Some things will need updating that I have noted and we'll take care of in other issues.

Testing

Preview docs using make doc-watch. Once merged, will publish

Issue(s)

Closes STOR-378.

@taddes taddes self-assigned this Jan 8, 2026
@taddes taddes marked this pull request as ready for review January 8, 2026 19:26
@taddes taddes force-pushed the docs/set-up-api-docs-STOR-378 branch 2 times, most recently from 1004a57 to 94270fd Compare January 8, 2026 20:23
@taddes taddes requested review from chenba and pjenvey January 8, 2026 20:50
@chenba
Copy link
Collaborator

chenba commented Jan 8, 2026

Thanks @taddes. I was able to preview the docs locally. From clicking around I found that

  • all the Syncstorage api pages 404 (and there's a typo in the title with "Styncstorage")
  • some rendering/formatting issue with the Setup section of the MdBook Notes page

@taddes
Copy link
Collaborator Author

taddes commented Jan 8, 2026

Thanks @taddes. I was able to preview the docs locally. From clicking around I found that

* all the Syncstorage api pages 404 (and there's a typo in the title with "Styncstorage")

* some rendering/formatting issue with the Setup section of the MdBook Notes page

Thanks @chenba ! I forgot to remove syncstorage/ from the internal links at the API doc index, thanks for catching that. They shouldn't 404 now. Fixed the instructions in the setup section too 👍

@taddes taddes force-pushed the docs/set-up-api-docs-STOR-378 branch from 94270fd to f757863 Compare January 8, 2026 22:29
@@ -0,0 +1,9 @@
# Styncstorage API
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# Styncstorage API
# Syncstorage API

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I propose an ADR where we consider renaming the service to Styncstorage 😆

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

Copy link
Member

Choose a reason for hiding this comment

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

👃 🤢

@chenba
Copy link
Collaborator

chenba commented Jan 8, 2026

I know it's not from a change on this PR but the Batches Table in syncstorage-postgres-db.md is borked.
image

@taddes
Copy link
Collaborator Author

taddes commented Jan 8, 2026

I know it's not from a change on this PR but the Batches Table in syncstorage-postgres-db.md is borked. image

Good catch. Fixed the bork

@taddes taddes requested a review from chenba January 9, 2026 00:14
@taddes taddes force-pushed the docs/set-up-api-docs-STOR-378 branch from 86bab03 to 198941e Compare January 9, 2026 14:42
@@ -0,0 +1,50 @@
<a id="howto_configure_tls"></a>

# Configure your Sync server for TLS
Copy link
Collaborator

Choose a reason for hiding this comment

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

We probably don't need this anymore?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yeah, I've got it on the books as something to potentially cull. Thx

Copy link
Collaborator

@chenba chenba left a comment

Choose a reason for hiding this comment

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

📖

@taddes taddes merged commit 77868a3 into master Jan 9, 2026
17 checks passed
@taddes taddes deleted the docs/set-up-api-docs-STOR-378 branch January 9, 2026 15:38
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.

4 participants