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

Adds functionality to reduce repeated problems in the problems ui #145

Merged
merged 1 commit into from
Sep 3, 2023

Conversation

bomoko
Copy link
Contributor

@bomoko bomoko commented Aug 8, 2023

Currently in the problems list we're seeing repeated problems from services that share the same base images etc.

This PR cleans up this display by reducing repeated problems to a single problem with services concatenated with commas

@bomoko bomoko requested a review from DaveDarsa August 8, 2023 18:18
// Duplicates are any items with the same source and identifier (multiple sources could report the same cve, but perhaps with different data)
// it appends the duplicated service names.
const reduceProblemsDuplicatedByService = problems => {
let reduceProblemsByService = new Map();
Copy link
Contributor

Choose a reason for hiding this comment

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

thumbs up for maps!

@tobybellwood tobybellwood added this to the 2.16.x milestone Sep 1, 2023
@DaveDarsa DaveDarsa merged commit 1fc1b5b into main Sep 3, 2023
@DaveDarsa DaveDarsa deleted the feature/merge_problems_by_service branch September 3, 2023 11:41
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.

3 participants