Skip to content

Fixed display data for the ReportsPage.js#1185

Open
mbusch3 wants to merge 2 commits into
ucfopen:form-ui-changesfrom
mbusch3:report-data-display
Open

Fixed display data for the ReportsPage.js#1185
mbusch3 wants to merge 2 commits into
ucfopen:form-ui-changesfrom
mbusch3:report-data-display

Conversation

@mbusch3
Copy link
Copy Markdown
Contributor

@mbusch3 mbusch3 commented May 11, 2026

Fixes multiple issues with the Reports page:

  • Previously, the chart and table that show barriers over time only showed the TOTAL count of issues, so when issues got fixed, the numbers didn't go down.
  • Also fixed an issue where the "Accessibility Barriers Remaining" table had an inaccurate number for Barriers Resolved. The number was from the initial scan, not updated after the Report.js's analyzeReport function.

@Ishfaq-code Ishfaq-code self-requested a review May 12, 2026 18:51
@dmols dmols self-requested a review May 15, 2026 12:27
Copy link
Copy Markdown
Contributor

@dmols dmols left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change looks good; the issue count is reflected appropriately now in the reports screen. Not sure if it's already in the target branch or not, but worth noting this branch also includes the fullscreen UDOIT feature. Will merge after getting an additional review.

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