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.

Sign up for GitHub

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

Jump to bottom

feat: display touchTs in grids for checklist, other assets, and reviews #1217

Merged

cd-rite merged 3 commits into main from checklist-touchTs

Feb 29, 2024

Member

csmig commented Feb 29, 2024

Adds timestamp values (ts, statusTs and touchTs) to the response body of GET /assets/{assetId}/checklists/{benchmarkId}/{revisionStr} when the format query parameter is json
Adds a grid column that displays touchTs in the Asset/STIG workspace for
- the Checklist grid
- the Other Assets grid
Adds a grid column that displays touchTs in the Collection/STIG workspace's Reviews grid
The grid columns are sortable but not filterable

csmig added 2 commits

February 28, 2024 19:03


          display touchTs in grids for checklist, reviews, and other assets

d073f93


          replace schema StringDateTime => StringDateTimeNullable

dff9df8

csmig requested a review from cd-rite

February 29, 2024 00:37


          move sprites and relabel in review.js

edccca2

sonarqubecloud bot commented Feb 29, 2024

Quality Gate passed for 'nuwcdivnpt_stig-manager-client'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

sonarqubecloud bot commented Feb 29, 2024

Quality Gate passed for 'nuwcdivnpt_stig-manager-api'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

cd-rite approved these changes

View reviewed changes

Collaborator

cd-rite left a comment

looks good!

cd-rite merged commit a7756b1 into main

10 checks passed

csmig deleted the checklist-touchTs branch

March 1, 2024 16:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet