Skip to content

Conversation

tdilauro
Copy link
Contributor

Description

  • Adds a library title-level report based on the LibraryAllTitleReportTable.
  • Adds a new /admin/reports/<report_key> endpoint through which a properly configured report may be requested by their keys.
  • Adds a generate_reports celery task that will be used to run reports requested through the endpoint.

Note: This PR depends on #2750. It should not be merged before that one has made it in.

Motivation and Context

Provides the report capability requested on PP-1888.

How Has This Been Tested?

  • Still needs tests

Checklist

  • N/A - I have updated the documentation accordingly.
  • All new and existing tests passed.

Copy link

codecov bot commented Sep 26, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.42%. Comparing base (e8f695b) to head (a998f67).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2763      +/-   ##
==========================================
+ Coverage   92.41%   92.42%   +0.01%     
==========================================
  Files         449      450       +1     
  Lines       42628    42683      +55     
  Branches     5955     5957       +2     
==========================================
+ Hits        39395    39451      +56     
+ Misses       2121     2120       -1     
  Partials     1112     1112              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Base automatically changed from feature/library-collection-report-support to main September 29, 2025 15:37
@tdilauro tdilauro force-pushed the feature/library-all-title-report branch from 1835c10 to 6ef0e73 Compare September 29, 2025 16:17
@tdilauro tdilauro marked this pull request as ready for review October 1, 2025 04:38
@tdilauro tdilauro requested a review from a team October 1, 2025 04:38
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.

1 participant