Skip to content

Commit

Permalink
Change "Classic Books" carousel on homepage to not link to /read (#10572
Browse files Browse the repository at this point in the history
)

* Change "Classic Books" carousel on homepage to not link to /read

/read is an old endpoint that sends you to the subject Accessible Books, which has like 12 books! Link to the default search results instead

---------

Co-authored-by: Mek <[email protected]>
  • Loading branch information
cdrini and mekarpeles authored Mar 11, 2025
1 parent a219065 commit 1eaec78
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion openlibrary/templates/home/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
$if not test:
$:render_template("books/custom_carousel", key="trending", books=get_trending_books(books_only=True, since_days=0, since_hours=24, minimum=3, sort_by_count=False), title=_('Trending Books'), url="/trending/daily", test=test, load_more={"url": "/trending/hours.json?hours=24&minimum=3&sort_by_count=false", "mode": "page", "limit": 18})

$:macros.QueryCarousel(query="ddc:8* first_publish_year:[* TO 1950] publish_year:[2000 TO *] NOT public_scan_b:false", title=_('Classic Books'), key="public_domain", url="/read", sort='random.hourly', use_cache=False)
$:macros.QueryCarousel(query="ddc:8* first_publish_year:[* TO 1950] publish_year:[2000 TO *] NOT public_scan_b:false", url="/search?q=ddc%3A8*+first_publish_year%3A%5B*+TO+1950%5D+publish_year%3A%5B2000+TO+*%5D+NOT+public_scan_b%3Afalse&mode=ebooks&has_fulltext=true" title=_('Classic Books'), key="public_domain", sort='random.hourly', use_cache=False)

$:render_template("home/custom_ia_carousel", title=_('Books We Love'), key="staff_picks", query='languageSorter:("English")', subject="openlibrary_staff_picks", sorts=["lending___last_browse desc"], limit=18, test=test)

Expand Down

0 comments on commit 1eaec78

Please sign in to comment.