Skip to content

Feature/sc 39979/add user download history to clark - #2299

Open
imrakeshgulla wants to merge 3 commits into
mainfrom
feature/sc-39979/add-user-download-history-to-clark
Open

imrakeshgulla wants to merge 3 commits into
mainfrom
feature/sc-39979/add-user-download-history-to-clark

Conversation

@imrakeshgulla

@imrakeshgulla imrakeshgulla commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Title

feature/sc-39979: Add user download history page


Description

Summary
Replaces the authenticated user's legacy Library page with a dedicated Download History page backed by the CLARK API GET /users/download-history endpoint.

Key Changes

  • Routing & Navigation: Added the /download-history route and set /library to automatically redirect to it. Updated primary navbar entries across desktop and mobile layouts to point to Download History.
  • API Integration: Integrated GET /users/download-history into LibraryService with support for paginated results via nextCursor.
  • UI & Workflows:
  • Replaced saved Library page elements (rating column, download button, delete modal, and removeFromLibrary logic) with a clean audit log view.
  • Added UI states for loading, empty history, error/retry, unavailable resources, and "Load More" pagination.
  • Row clicks now safely fetch current details and navigate to the learning object details page instead of triggering immediate downloads or treating entries as active saved items.

Verification & Testing

  • Type Check & Lint: Passed npx tsc -p src/tsconfig.app.json --noEmit and npx ng lint clark.
  • Formatting: Verified using npx prettier --check and git diff --check.
  • Specs: Added focused unit tests for LibraryService API route construction and LibraryComponent state management.
Screen.Recording.2026-09-14.at.12.40.54.PM.mov

@Cwagne17

Copy link
Copy Markdown
Contributor

Could you add a screenshot or video recording?

@imrakeshgulla
imrakeshgulla force-pushed the feature/sc-39979/add-user-download-history-to-clark branch from 3fe2d9b to 7b05dda Compare September 15, 2026 14:14
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