Skip to content

Download validation results, include timestamp and version #58

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

Merged
merged 10 commits into from
Apr 3, 2025

Conversation

mint-thompson
Copy link
Collaborator

Download validation results, include timestamp and version

Problem

Validator output lacks useful contextual information, such as time that the validator was run. Additionally, it is difficult to save results in a text-based format.

Solution

Add start time, end time, and hpt-validator version to web display of results. Add a button to download a results file. The results file contains the new contextual information above the list of errors.

Test Plan

Run the hpt-tool site locally and use the online validator. Confirm that start and end time are displayed as part of the results. Confirm that the current hpt-validator version is displayed as part of the results. This version should be 1.10.0 at time of writing. Use the new button to download the result file. Confirm that the result file contains the same information as displayed in the web results. Confirm that the downloaded result file is named based on the input file name.

Re-add previously existing download feature. Use the location header as
part of the results.
The result file name contains the input file name. The extension is
removed from the input file name. If the resulting length is trimmed to
170 characters so that the final file name is short enough to be opened
in Excel.
Copy link
Collaborator

@shaselton-usds shaselton-usds left a comment

Choose a reason for hiding this comment

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

Sprinkling feature joy everywhere!

@mint-thompson mint-thompson merged commit 60a8c56 into main Apr 3, 2025
4 checks passed
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