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

Change Report client UI to put all reports into the same drop-down #184

Closed
vteague opened this issue Aug 16, 2024 · 0 comments
Closed

Change Report client UI to put all reports into the same drop-down #184

vteague opened this issue Aug 16, 2024 · 0 comments

Comments

@vteague
Copy link
Member

vteague commented Aug 16, 2024

The UI for (IRV-upgraded) Report download currently has a rather clunky separate pair of buttons for getting the assertion reports (json and csv). It'd be nicer if those two were incorporated into the existing drop-down.

This would require dealing with two technical matters:

  1. Currently, colorado-rla calls out to raire for those reports, which is fine if the internet connection etc is working, but might be an annoyance if someone has turned it off since assertion generation. It'd perhaps be prudent to do them last, because unfortunately at the moment if one audit report fails, nothing after it seems to run.
  2. Currently, raire-service returns a zip of each thing (either a zip of all the csv files, one for each contest, or likewise for json). I guess if we do nothing we get a zip inside a zip, which probably works but isn't great. Perhaps the best thing to do is unzip the zip, incorporate it into the AuditReport folder as a subdirectory, and then zip the lot.
@vteague vteague moved this to Todo in CO IRV RLA Aug 16, 2024
@vteague vteague self-assigned this Aug 23, 2024
@vteague vteague moved this from Todo to Testing & Review in CO IRV RLA Aug 24, 2024
@vteague vteague closed this as completed Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
1 participant