Skip to content

Website: Add dashboard vulnerability exposure article#46485

Open
lukeheath wants to merge 3 commits into
mainfrom
lukeheath/dashboard-vuln-exposure-article
Open

Website: Add dashboard vulnerability exposure article#46485
lukeheath wants to merge 3 commits into
mainfrom
lukeheath/dashboard-vuln-exposure-article

Conversation

@lukeheath
Copy link
Copy Markdown
Member

@lukeheath lukeheath commented May 29, 2026

Summary

  • Adds articles/dashboard-vulnerability-exposure.md explaining which software the dashboard's "Vulnerability exposure" chart covers and why coverage is curated (reach/risk + bitmap-storage performance).
  • Repoints the existing /learn-more-about/vulnerability-exposure-cves redirect from a pinned GitHub line range in charts.go to the new article. The dashboard tooltip URL is unchanged, so older Fleet builds continue to work.

Test plan

  • npm run build-for-prod in website/ succeeds (markdown pipeline compiles the new article).
  • Hover the "Vulnerability exposure" tooltip on the dashboard; clicking "other common software" lands on the new article.
  • Verify the article renders cleanly on fleetdm.com (table, meta tags, links).

Summary by CodeRabbit

  • Bug Fixes
    • Updated the vulnerability exposure "learn more" redirect to point to an internal article page instead of an external resource.

Review Change Stack

Documents which software the dashboard's "Vulnerability exposure" chart
covers and explains why coverage is curated rather than exhaustive.
Repoints the existing /learn-more-about/vulnerability-exposure-cves
redirect from a pinned GitHub line range to the new article so the
existing tooltip link keeps working.
@lukeheath lukeheath self-assigned this May 29, 2026
Comment thread articles/dashboard-vulnerability-exposure.md Outdated
Comment thread articles/dashboard-vulnerability-exposure.md Outdated
@lukeheath lukeheath marked this pull request as ready for review May 29, 2026 19:28
Copilot AI review requested due to automatic review settings May 29, 2026 19:28
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@lukeheath
Copy link
Copy Markdown
Member Author

@sgress454 @AndreyKizimenko Please check this article for accuracy. Thanks!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new fleetdm.com article explaining the dashboard's "Vulnerability exposure" chart coverage, and repoints the existing /learn-more-about/vulnerability-exposure-cves redirect from a pinned GitHub source link to the new article.

Changes:

  • New article articles/dashboard-vulnerability-exposure.md describing curated software list and rationale (reach/risk + bitmap performance).
  • Redirect in website/config/routes.js updated to point at the new article.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
articles/dashboard-vulnerability-exposure.md New guide article with category, author, and meta tags.
website/config/routes.js Updates /learn-more-about/vulnerability-exposure-cves redirect to the new article.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 29, 2026

Walkthrough

The PR updates a single route redirect in website/config/routes.js at line 1365. The GET /learn-more-about/vulnerability-exposure-cves shortlink destination is changed from an external GitHub blob URL to an internal article path at /articles/dashboard-vulnerability-exposure. This consolidates the vulnerability exposure documentation reference to a site-owned resource.

Possibly related PRs

  • fleetdm/fleet#45079: Previously updated the same GET /learn-more-about/vulnerability-exposure-cves redirect route in website/config/routes.js to a different external destination.
🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The PR description lacks required checklist items and sections specified in the template, making it incomplete. Add the missing checklist sections from the template (changes files, security validation, testing, database migrations, etc.) and mark applicable items as completed or delete non-applicable ones.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Website: Add dashboard vulnerability exposure article' clearly summarizes the main change - adding a new article to the website about dashboard vulnerability exposure.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch lukeheath/dashboard-vuln-exposure-article

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

5 participants