Skip to content

UI - Fix handling of really long cert names#38116

Merged
jacobshandling merged 1 commit intomainfrom
38107-truncate-listed-certs
Jan 9, 2026
Merged

UI - Fix handling of really long cert names#38116
jacobshandling merged 1 commit intomainfrom
38107-truncate-listed-certs

Conversation

@jacobshandling
Copy link
Copy Markdown
Contributor

Related issue: Resolves #38107

Screen.Recording.2026-01-09.at.11.20.08.AM.mov
  • QA'd all new/changed functionality manually

@jacobshandling jacobshandling requested a review from a team as a code owner January 9, 2026 19:24
Copy link
Copy Markdown
Member

@getvictor getvictor left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.86%. Comparing base (2c962f0) to head (4a1e14a).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #38116   +/-   ##
=======================================
  Coverage   65.86%   65.86%           
=======================================
  Files        2392     2392           
  Lines      190790   190790           
  Branches     8477     8476    -1     
=======================================
  Hits       125661   125661           
  Misses      53707    53707           
  Partials    11422    11422           
Flag Coverage Δ
frontend 54.15% <ø> (ø)

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jacobshandling jacobshandling merged commit 6f4845e into main Jan 9, 2026
17 checks passed
@jacobshandling jacobshandling deleted the 38107-truncate-listed-certs branch January 9, 2026 19:57
jacobshandling added a commit that referenced this pull request Jan 9, 2026
<!-- Add the related story/sub-task/bug number, like Resolves #123, or
remove if NA -->
**Related issue:** Resolves #38107 


https://github.com/user-attachments/assets/86f509bf-da0a-45bf-ba16-3ea2b19800e1


- [x] QA'd all new/changed functionality manually
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.

Long Certificate names do not truncate on the Certificates page

2 participants