Skip to content

docs: fix obsolete OPDS and tooling references - #13854

Merged
mekarpeles merged 1 commit into
internetarchive:masterfrom
harshitethic:docs/fix-obsolete-opds-pam-references
Oct 8, 2026
Merged

mekarpeles merged 1 commit into
internetarchive:masterfrom
harshitethic:docs/fix-obsolete-opds-pam-references

Conversation

@harshitethic

Copy link
Copy Markdown
Contributor

Summary\n- Replace an absent internal OPDS document reference with a link to the checked-in OPDS README.\n- Clarify that the historical carousel-batch plan was superseded by the actual OPDS fix.\n- Remove the undocumented .pam migration claim from contributor guidance.\n\n## Validation\n- git diff --check\n- git grep -n -E "opds-system\\.md|\\.pam" -- docs/ (no remaining matches)\n- Confirmed docs/opds/README.md exists.\n\nFixes #13852.

@harshitethic
harshitethic force-pushed the docs/fix-obsolete-opds-pam-references branch from 49d9524 to 795ae41 Compare October 8, 2026 02:27
@openlibrary-bot

Copy link
Copy Markdown
Collaborator

Thanks for the PR, @harshitethic, and welcome to Open Library's contributor community! πŸŽ‰

The linked issue hasn't been triaged yet β€” triage happens on Mondays and Fridays. There are currently 63 open non-draft PRs ahead of yours.

PR triage checklist (maintainers / Pierre)
  • PR description β€” not empty; explains what the change does and how to verify it
  • References an issue β€” PR body contains a #NNN reference
    • Linked issue is triaged β€” has a Priority: * label (not just Needs: Triage)
    • Linked issue is assigned β€” has at least one assignee
  • Commit history clean β€” no WIP/fixup/conflict noise; commit messages are meaningful
  • CI passing β€” no failing check-runs
  • Test cases present β€” if the change touches substantive logic, test coverage exists or is explained
  • Proof of testing β€” PR body includes a description of what was tested, a screenshot, or a video

Note

This comment was automatically generated by Pierre, Open Library's PR first-touch assistant, on behalf of @mekarpeles. This is NOT a code review or sign-off.

@mekarpeles
mekarpeles merged commit 744d07c into internetarchive:master Oct 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants