Skip to content

refactor: Clean up unused templates#11604

Merged
cdrini merged 4 commits intointernetarchive:masterfrom
lokesh:dead-code
Dec 18, 2025
Merged

refactor: Clean up unused templates#11604
cdrini merged 4 commits intointernetarchive:masterfrom
lokesh:dead-code

Conversation

@lokesh
Copy link
Copy Markdown
Collaborator

@lokesh lokesh commented Dec 18, 2025

Removes unused files:

  • openlibrary/templates/books/show.html
  • openlibrary/templates/books/works-show.html
  • openlibrary/templates/covers/book_cover_single_edition.html
  • openlibrary/templates/covers/book_cover_work.html

Testing

I did the following to test:

  • ran tests locally: docker compose run --rm home make test
  • visited key pages: Homepage, book (aka works) page, edition page, search results, account page

Stakeholders

@mekarpeles @cdrini

Copy link
Copy Markdown
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Lgtm! Gave a whirl on testing, and everything seemed to render normally.

@cdrini cdrini merged commit 43e1a7e into internetarchive:master Dec 18, 2025
5 checks passed
mystic-06 pushed a commit to mystic-06/openlibrary that referenced this pull request Dec 24, 2025
* remove: delete unused book cover templates for single edition and work

* remove: eliminate unused references to the single edition book cover template in legacy CSS

* remove: delete unused book cover templates for single edition and work, and clean up legacy CSS references
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