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(dashboard): display additional portfolio statistics #709

Merged
merged 5 commits into from
Jan 30, 2024

Conversation

setchy
Copy link
Contributor

@setchy setchy commented Jan 28, 2024

Description

Display the following additional portfolio statistics

  • Vulnerable Projects
  • Vulnerable Components
  • Policy Violations
  • License Violations
  • Operational Violations
  • Security Violations

Addressed Issue

Display additional Portfolio Statistics.

Additional Details

Checklist

@setchy setchy changed the title refactor(dashboard): swap vulnerabilities for violations. refactor(dashboard): display violations metric in portfolio statistics Jan 28, 2024
@nscuro
Copy link
Member

nscuro commented Jan 28, 2024

I'm thinking rather than replacing the "Portfolio Vulnerabilities" number, should the policy violations number be displayed in addition to it?

I agree it's odd at first that vuln count is displayed twice, however once users scroll to the bottom of the page where "Portfolio Statistics" is located, the top left number is no longer visible:
image

@setchy
Copy link
Contributor Author

setchy commented Jan 28, 2024

Great feedback @nscuro - I've pushed an update that keeps both, and added a few more datapoints to fill out the space

Screenshot 2024-01-28 at 5 28 46 PM

@setchy setchy changed the title refactor(dashboard): display violations metric in portfolio statistics refactor(dashboard): display additional portfolio statistics Jan 28, 2024
@setchy setchy changed the title refactor(dashboard): display additional portfolio statistics feat(dashboard): display additional portfolio statistics Jan 29, 2024
@nscuro nscuro added the enhancement New feature or request label Jan 30, 2024
@nscuro nscuro merged commit e18c7ee into DependencyTrack:master Jan 30, 2024
9 checks passed
@setchy setchy deleted the refactor/portfolio-statistics branch February 21, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants