Skip to content

Inject Reports Overview progress services via constructor - #8210

Draft
donnapep wants to merge 2 commits into
hpps-reports-courses-completions-progressfrom
hpps-reports-constructor-injection
Draft

Inject Reports Overview progress services via constructor#8210
donnapep wants to merge 2 commits into
hpps-reports-courses-completions-progressfrom
hpps-reports-constructor-injection

Conversation

@donnapep

@donnapep donnapep commented Aug 25, 2026

Copy link
Copy Markdown
Member

SEN-83: HPPS: Reports Overview aggregates → table-aware

Part of the Reports → Overview HPPS effort. Stacked on hpps-reports-courses-completions-progress.

Internal cleanup only — no user-facing change.

Proposed Changes

  • Inject the progress query services into the Reports Overview list tables and services via the constructor (with a factory fallback) to match the repository's dependency convention.
  • Move the use imports above the ABSPATH guard in the two list tables for consistency with the sibling lessons list table.
  • No columns or values change.

Testing Instructions

  • With HPPS enabled, open Reports → Overview, view the Students and Courses tabs, and confirm both render exactly as before (no value or column changes).
  • Export each tab's CSV and confirm the output is unchanged.

@donnapep donnapep added this to the 4.26.4 milestone Aug 25, 2026
@donnapep donnapep added the No Changelog No change log entry needed label Aug 25, 2026
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

WordPress Playground Preview

The changes in this pull request can previewed and tested using a WordPress Playground instance.

Open WordPress Playground Preview

donnapep and others added 2 commits August 25, 2026 11:25
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Match the ordering in class-sensei-reports-overview-list-table-lessons.php.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@donnapep
donnapep force-pushed the hpps-reports-courses-completions-progress branch from 60acda6 to fc9a88a Compare August 25, 2026 15:26
@donnapep
donnapep force-pushed the hpps-reports-constructor-injection branch from 92e146d to 587def5 Compare August 25, 2026 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Changelog No change log entry needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant