Skip to content

fix(ui): Add UI changes for Not Available NVDCVSS - #23853

Open
vjwilson wants to merge 1 commit into
masterfrom
vjwilson/ROX-30599-ui-missing-CVSS-score
Open

vjwilson wants to merge 1 commit into
masterfrom
vjwilson/ROX-30599-ui-missing-CVSS-score

Conversation

@vjwilson

@vjwilson vjwilson commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Description

Handle unavailable CVSS and NVD CVSS scores in vulnerability tables, widgets, and tooltips without throwing during numeric formatting. Shared formatting helpers preserve numeric zero and the existing formatting for available scores.

Extracted UI prerequisite for #23561. This PR contains only changes under ui/apps/platform; the backend, schema, and report changes remain in #23561.

User-facing documentation

The combined change's changelog entry remains in #23561. No separate documentation update is needed.

Testing and quality

  • the change is production ready: the change is GA, or otherwise the functionality is gated by a feature flag
  • CI results are inspected

Automated testing

  • added unit tests
  • added e2e tests
  • added regression tests
  • added compatibility tests
  • modified existing tests

How I validated my change

  • The extracted branch passes 62 affected unit tests and 19 component tests across six specs.
  • TypeScript, ESLint on all changed files, and diff checks pass.
  • The combined change was deployed and manually validated before extraction. All 19 extracted UI files match the original branch byte for byte.

Verified that this new UI still works even with the existing backend, before the corresponding backend changes are merged

Screenshot 2026-11-03 at 6 49 36 PM

@vjwilson
vjwilson requested a review from a team as a code owner November 3, 2026 20:29
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

🚀 Build Images Ready

Images are ready for commit 543dd48. To use with deploy scripts:

export MAIN_IMAGE_TAG=5.1.x-462-g543dd48419

@codecov

codecov Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 52.57%. Comparing base (cc3d9a7) to head (543dd48).
⚠️ Report is 5 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #23853      +/-   ##
==========================================
- Coverage   52.61%   52.57%   -0.05%     
==========================================
  Files        2836     2836              
  Lines      177975   177975              
==========================================
- Hits        93642    93569      -73     
- Misses      76140    76192      +52     
- Partials     8193     8214      +21     
Flag Coverage Δ
go-unit-tests 52.57% <ø> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vjwilson vjwilson changed the title bug(ui): Add UI changes for Not Available NVDCVSS fix(ui): Add UI changes for Not Available NVDCVSS Nov 3, 2026

This branch has not been deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant