Skip to content

Feat: Added Continue Reading Carousel on Homepage #11562

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
KrishnaShuk wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Feat: Added Continue Reading Carousel on Homepage #11562

KrishnaShuk wants to merge 1 commit into from
+98 −2

Conversation

@KrishnaShuk
Copy link
Contributor

@KrishnaShuk KrishnaShuk commented Dec 7, 2025 •
edited
Loading

Closes #11371

This PR implements the initial version of the "Continue Reading" carousel on the homepage. It displays the currently active loans for logged-in users.

Technical

  • Backend (partials.py): Added ContinueReadingPartial handler.

    • Fetches the current user's active loans using get_loans_of_user.
    • Formats the book data for the custom_carousel template.
    • Returns an empty response if the user is not logged in or has no loans.

  • Frontend (index.html): Added a lazy-carousel placeholder.

    • The placeholder is always rendered in the cached homepage template.
    • Client-side JavaScript fetches the content via /partials.json?_component=ContinueReading.
    • This approach ensures user-specific content can be displayed on the globally cached homepage.

  • Pending Work

    • Loan History
    • Availability Buttons

Testing

  • Verified locally using mock loans.
  • Confirmed that the carousel does not appear for logged-out users.
  • Confirmed that the carousel loads asynchronously without blocking the page load.

Screenshot

image

Stakeholders

@KrishnaShuk KrishnaShuk changed the title Initial Commit Feat: Added Continue Reading Carousel on Homepage Dec 7, 2025
@KrishnaShuk
feat: Add Continue Reading carousel to homepage
2b6ea0c 
- Added ContinueReadingPartial in partials.py to fetch user's active loans
- Added lazy-carousel placeholder in index.html for logged-in users
- Updated i18n messages.pot for new strings
@KrishnaShuk KrishnaShuk force-pushed the 11371/feat/ContinueReadingCarousel branch from 2186b76 to 2b6ea0c Compare December 10, 2025 06:46
@KrishnaShuk KrishnaShuk marked this pull request as ready for review December 10, 2025 07:13
@mekarpeles mekarpeles self-assigned this Dec 15, 2025
@KrishnaShuk
Copy link
Contributor Author

KrishnaShuk commented Dec 16, 2025

@mekarpeles need your brief guidance on the completion of this feature. I have used mock loan books for accessing the feature as archive api key is not provided.
I need you to check the present logic so i can further complete this.

@github-actions github-actions bot added the Needs: Response Issues which require feedback from lead label Dec 17, 2025
@mekarpeles mekarpeles assigned jimchamp and unassigned mekarpeles Dec 17, 2025
@mekarpeles
Copy link
Member

mekarpeles commented Dec 17, 2025

Looks like the feature is headed in the right direction. It should only show up if the patron is logged in and has loans. Since this code is touching partials, I am handing it off to @jimchamp

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jimchamp jimchamp

Labels

Needs: Response Issues which require feedback from lead

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Continue Reading Carousel on Homepage

3 participants

@KrishnaShuk @mekarpeles @jimchamp