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

fix: correctly calculate audited violations and audited findings percentages #704

Merged

Conversation

setchy
Copy link
Contributor

@setchy setchy commented Jan 25, 2024

Description

Audited Violations Percentage and Audited Findings Percentage were being incorrectly calculated due to a typo.

Addressed Issue

Uses the correct properties to calculate the percentages

Additional Details

Using mocked data

Before
Screenshot 2024-01-26 at 1 25 51 PM

After
Screenshot 2024-01-26 at 1 25 04 PM

Checklist

@setchy setchy force-pushed the fix/audited-violations-percentage branch from 3d4eb1b to 28c4f18 Compare January 25, 2024 15:39
Signed-off-by: Adam Setch <adam.setch@outlook.com>
Signed-off-by: Adam Setch <adam.setch@outlook.com>
@setchy setchy force-pushed the fix/audited-violations-percentage branch from 080e69d to 5e5a004 Compare January 26, 2024 18:27
@setchy setchy changed the title fix: correctly calculate audited violations percentage fix: correctly calculate audited violations and audited findings percentages Jan 26, 2024
Copy link
Member

@nscuro nscuro left a comment

Choose a reason for hiding this comment

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

Good spot, and thanks for the PR!

@nscuro nscuro added the defect Something isn't working label Jan 27, 2024
@nscuro nscuro added this to the 4.11 milestone Jan 27, 2024
@nscuro nscuro merged commit 151c4b3 into DependencyTrack:master Jan 27, 2024
9 checks passed
@setchy setchy deleted the fix/audited-violations-percentage branch February 21, 2024 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants