Skip to content

Remove summary table from report dashboard - #1523

Open
jazairi wants to merge 1 commit into
mainfrom
etd-623
Open

Remove summary table from report dashboard#1523
jazairi wants to merge 1 commit into
mainfrom
etd-623

Conversation

@jazairi

@jazairi jazairi commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Why these changes are being introduced:

The report summary table is extremely inefficient.

Relevant ticket(s):

How this addresses that need:

This removes the summary table, so the report
index view just lists links to the individual
reports.

Stakeholders have noted that the summary tables
are useful, so this is an intermediary step
toward a functional version.

Side effects of this change:

  • Link sidebar is now redundant, but we intend to
    reintroduce the summary table in a more efficient
    form, so it makes sense to leave it in for now.
  • The report dashboard is now a visual design
    abomination, but that feels better than a million
    N+1 queries.

Developer

  • All new ENV is documented in README
  • All new ENV has been added to Heroku Pipeline, Staging and Prod
  • ANDI or Wave has been run in accordance to
    our guide and
    all issues introduced by these changes have been resolved or opened as new
    issues (link to those issues in the Pull Request details above)
  • Stakeholder approval has been confirmed (or is not needed)

Code Reviewer

  • The commit message is clear and follows our guidelines
    (not just this pull request message)
  • There are appropriate tests covering any new functionality
  • The documentation has been updated or is unnecessary
  • The changes have been verified
  • New dependencies are appropriate or there were no changes

Requires database migrations?

NO

Includes new or updated dependencies?

NO

@jazairi

jazairi commented Jul 29, 2026

Copy link
Copy Markdown
Contributor Author

I'd like to check with Wilder about whether they use the report summary, and they're out sick today. Will tag for review when I have confirmation that the table isn't needed.

@mitlib
mitlib temporarily deployed to thesis-submit-pr-1523 July 29, 2026 17:48 Inactive
@coveralls

coveralls commented Jul 29, 2026

Copy link
Copy Markdown

Coverage Status

coverage: 98.286% (-0.002%) from 98.288% — etd-623 into main

Why these changes are being introduced:

The report summary table is extremely inefficient.

Relevant ticket(s):

- [ETD-623](https://mitlibraries.atlassian.net/browse/ETD-623)

How this addresses that need:

This removes the summary table, so the report
index view just lists links to the individual
reports.

Stakeholders have noted that the summary tables
are useful, so this is an intermediary step
toward a functional version.

Side effects of this change:

- Link sidebar is now redundant, but we intend to
reintroduce the summary table in a more efficient
form, so it makes sense to leave it in for now.
- The report dashboard is now a visual design
abomination, but that feels better than a million
N+1 queries.
@jazairi

jazairi commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

Mikki would like the report summary to include the past four terms. This is ticketed here, but I think we should still land this PR in the meantime so the route isn't broken.

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.

3 participants