Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(admin): Reports Screen Content Revamp #4028

Merged
merged 7 commits into from
Oct 4, 2023
Merged

Conversation

adghayes
Copy link
Collaborator

@adghayes adghayes commented Oct 4, 2023

Overview

Closes #3992. Branches from #4019. Removes our lists of reports from the reports screen in favor of a few preset reports based on known customer needs. It's still possible to make all of the reports removed (and way more) with the report builder.

Demo Video or Screenshot

All Screens

all-pages-reports-revamp.mov

Reports Screen (Before)

Screen Shot 2023-10-04 at 2 59 52 PM Screen Shot 2023-10-04 at 3 07 47 PM

Full Election Tally Report Screen (Before)

Screen Shot 2023-10-04 at 3 00 46 PM

Testing Plan

High automated testing level on reporting screens, and manual pass through.

Checklist

  • I have added logging where appropriate to any new user actions, system updates such as file reads or storage writes, or errors introduced.
  • I have added a screenshot and/or video to this PR to demo the change
  • I have added the "user_facing_change" label to this PR to automate an announcement in #machine-product-updates

@adghayes adghayes changed the title Drew/reports screen revamp Reports Screen Content Revamp Oct 4, 2023
@adghayes adghayes marked this pull request as ready for review October 4, 2023 12:05
@adghayes adghayes requested a review from a team as a code owner October 4, 2023 12:05
@adghayes adghayes requested review from kshen0 and kofi-q and removed request for a team and kshen0 October 4, 2023 12:05
@adghayes adghayes changed the title Reports Screen Content Revamp feat(admin): Reports Screen Content Revamp Oct 4, 2023
Base automatically changed from drew/ballot-count-report-builder to main October 4, 2023 21:24
@adghayes adghayes force-pushed the drew/reports-screen-revamp branch from a57d853 to 8292724 Compare October 4, 2023 21:26
@adghayes adghayes merged commit 855c1d6 into main Oct 4, 2023
@adghayes adghayes deleted the drew/reports-screen-revamp branch October 4, 2023 21:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

VxAdmin Reports: Rearrange + Revamp Reports Screen
2 participants