Skip to content

feat(heureka): adds vulnerability details panel#1117

Merged
hodanoori merged 11 commits intomainfrom
heureka-vulnerability-panel
Aug 7, 2025
Merged

feat(heureka): adds vulnerability details panel#1117
hodanoori merged 11 commits intomainfrom
heureka-vulnerability-panel

Conversation

@hodanoori
Copy link
Copy Markdown
Contributor

@hodanoori hodanoori commented Aug 5, 2025

Summary

This PR adds the vulnerability details panel to be displayed as soon as a user clicks on each vulnerability in the list. The panel shows the list of support groups and services which are having the selected vulnerability. In order to see more details for each service in the panel, if the user clicks on each service in the list, it will navigate to the service details page.

Changes Made

  • VulnerabilityDetailsPanel, along with all required components (such as VulnerabilityServicesList and VulnerabilitySupportGroups), is added to the application.

Related Issues

Screenshots (if applicable)

E1338B63-55FE-4856-9144-182AB505CD60

Testing Instructions

  1. pnpm i
  2. pnpm TASK

Checklist

  • I have performed a self-review of my code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • New and existing unit tests pass locally with my changes.
  • I have made corresponding changes to the documentation (if applicable).
  • My changes generate no new warnings or errors.
  • I have created a changeset for my changes.

PR Manifesto

Review the PR Manifesto for best practises.

@hodanoori hodanoori self-assigned this Aug 5, 2025
@hodanoori hodanoori requested a review from a team as a code owner August 5, 2025 14:45
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Aug 5, 2025

🦋 Changeset detected

Latest commit: f85d159

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@cloudoperators/juno-app-heureka Minor
@cloudoperators/juno-app-greenhouse Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Aug 5, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-08-07 13:37 UTC

Copy link
Copy Markdown
Contributor

@TilmanHaupt TilmanHaupt left a comment

Choose a reason for hiding this comment

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

Good so far - i guess my comments can be partially rejected but take a look.

Comment thread apps/heureka/src/api/fetchVulnerabilities.tsx
Copy link
Copy Markdown
Contributor

@taymoor89 taymoor89 left a comment

Choose a reason for hiding this comment

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

Great job !!!

  • Let's try to remove anys
  • Let's add changeset

Comment thread apps/heureka/src/components/Vulnerabilities/utils.ts Outdated
Comment thread apps/heureka/src/components/Vulnerabilities/utils.ts Outdated
Comment thread apps/heureka/src/components/Vulnerabilities/utils.ts Outdated
Comment thread apps/heureka/src/components/Vulnerabilities/utils.ts Outdated
TilmanHaupt
TilmanHaupt previously approved these changes Aug 7, 2025
Copy link
Copy Markdown
Contributor

@taymoor89 taymoor89 left a comment

Choose a reason for hiding this comment

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

LGTM!

@hodanoori hodanoori merged commit 6ce69e4 into main Aug 7, 2025
16 checks passed
@hodanoori hodanoori deleted the heureka-vulnerability-panel branch August 7, 2025 13:37
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