Update software-management.ejs - #44812
Conversation
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
Pull request overview
Updates the website’s Software management marketing page to emphasize Fleet’s automated patching and compliance messaging, including refreshed hero copy and re-ordered carousel content to match the new narrative.
Changes:
- Updated hero and bottom CTA headline to “Patch faster. Stay compliant.”
- Reordered hero carousel slide headings and swapped slide images to align with the new slide sequence.
- Rewrote the “Automate patch management”, “Mitigate CVEs automatically”, and “Report what matters” sections to describe auto-patching capabilities and outcomes.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| <div purpose="feature-text"> | ||
| <h3>Automate patch management</h3> | ||
| <p>Take the guesswork out of keeping devices up-to-date and secure by asking yes or no questions tied to automations.</p> | ||
| <p>Fleet’s policy engine continuously evaluates every managed device using osquery - installed software versions, configuration state, and system metrics. When a device falls out of compliance, Fleet automatically installs the correct software. No ticket. No manual intervention.</p> |
| <div purpose="feature-text" class="d-flex flex-column"> | ||
| <h3>Mitigate CVEs automatically</h3> | ||
| <p>Build automations that close tickets or mitigate vulnerabilities by updating or removing software and running custom scripts.</p> | ||
| <p>When a vulnerability is disclosed, Fleet already knows which devices are exposed. Automatically update or remove vulnerable software and run custom scripts - before a ticket is ever created.</p> |
| <div purpose="feature-text" class="d-flex flex-column"> | ||
| <h3>Report what matters</h3> | ||
| <p>Let's face it, most built-in graphs leave you wanting more. Report MTTR and any other custom metrics exactly the way you want to using fresh data from real computers.</p> | ||
| <p>See which devices have out-of-date software, which vulnerabilities they're exposed to, and exactly how long they've been at risk - down to the hour.</p> |
WalkthroughThis pull request updates the software management page template with revised messaging and feature descriptions. The hero section heading changes from "Manage software consistently" to "Patch faster. Stay compliant." A new carousel slide titled "Automate patch management" is inserted and reordered before "Self-service on any OS." Carousel images are repositioned accordingly. Feature sections are updated, including expanded descriptions for patch automation with new checklist items emphasizing hourly policy runs and staged deployments. The CVE mitigation, on-demand data, and reporting sections receive refreshed copy. The bottom CTA heading is also aligned with the new hero messaging. Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 inconclusive)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
website/views/pages/software-management.ejs (1)
152-152: ⚡ Quick winAlign this heading with the feature-switch label.
The switch still says “On-demand data,” but this panel now headlines “Ground-truth data from every device.” That mismatch makes it feel like the wrong panel opened. Consider keeping the control label and panel heading in sync.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@website/views/pages/software-management.ejs` at line 152, The panel heading "Ground-truth data from every device" is out of sync with the feature-switch label "On-demand data"; update one of them so both labels match (either change the heading text to "On-demand data" or change the switch label to "Ground-truth data from every device") and ensure any associated aria-labels or data-test attributes that reference the strings are updated too (look for the literal strings "Ground-truth data from every device" and "On-demand data" in the template and update them consistently).
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@website/views/pages/software-management.ejs`:
- Around line 40-43: Update the two carousel image alt attributes so they are
meaningful to screen readers: replace the generic "Second slide" and "Third
slide" alt text on the <img> elements whose src values end with
"software-management-feature-slide-3-2144x960@2x.png" and
"software-management-feature-slide-2-2144x960@2x.png" with concise descriptive
text that summarizes each slide's visible content (or use an empty alt="" if the
adjacent heading/paragraph already conveys the same information and the images
are purely decorative).
- Around line 25-28: The slide picker is a plain div and not
keyboard-accessible; replace the element with a semantic button (change the
element having purpose="slide-text" and data-target="#heroCarousel"
data-slide-to="2" from <div> to <button type="button">), preserve the existing
data-* attributes and classes, add accessible attributes like
aria-controls="heroCarousel" and aria-label (or visually hidden text) and ensure
aria-pressed/aria-selected updates with selection state; also update any JS/CSS
that queries purpose="slide-text" to continue working with the button element.
---
Nitpick comments:
In `@website/views/pages/software-management.ejs`:
- Line 152: The panel heading "Ground-truth data from every device" is out of
sync with the feature-switch label "On-demand data"; update one of them so both
labels match (either change the heading text to "On-demand data" or change the
switch label to "Ground-truth data from every device") and ensure any associated
aria-labels or data-test attributes that reference the strings are updated too
(look for the literal strings "Ground-truth data from every device" and
"On-demand data" in the template and update them consistently).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 9fd105f7-d6c1-4ca6-a30e-313f4c7c958e
📒 Files selected for processing (1)
website/views/pages/software-management.ejs
| <div purpose="slide-text" data-target="#heroCarousel" data-slide-to="2"> | ||
| <h5>Self-service on any OS</h5> | ||
| <p>Enable end users on every operating system and reduce IT burden with pre-approved software.</p> | ||
| </div> |
There was a problem hiding this comment.
Use a real control for the new carousel selector.
This slide picker is still a plain div, so it won't be keyboard-focusable or operable without a mouse. Please render the carousel heading selectors as buttons (or add equivalent keyboard semantics) while touching this markup.
Suggested markup
- <div purpose="slide-text" data-target="#heroCarousel" data-slide-to="2">
+ <button type="button" purpose="slide-text" data-target="#heroCarousel" data-slide-to="2">
<h5>Self-service on any OS</h5>
<p>Enable end users on every operating system and reduce IT burden with pre-approved software.</p>
- </div>
+ </button>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <div purpose="slide-text" data-target="#heroCarousel" data-slide-to="2"> | |
| <h5>Self-service on any OS</h5> | |
| <p>Enable end users on every operating system and reduce IT burden with pre-approved software.</p> | |
| </div> | |
| <button type="button" purpose="slide-text" data-target="#heroCarousel" data-slide-to="2"> | |
| <h5>Self-service on any OS</h5> | |
| <p>Enable end users on every operating system and reduce IT burden with pre-approved software.</p> | |
| </button> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@website/views/pages/software-management.ejs` around lines 25 - 28, The slide
picker is a plain div and not keyboard-accessible; replace the element with a
semantic button (change the element having purpose="slide-text" and
data-target="#heroCarousel" data-slide-to="2" from <div> to <button
type="button">), preserve the existing data-* attributes and classes, add
accessible attributes like aria-controls="heroCarousel" and aria-label (or
visually hidden text) and ensure aria-pressed/aria-selected updates with
selection state; also update any JS/CSS that queries purpose="slide-text" to
continue working with the button element.
| <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Second slide"> | ||
| </div> | ||
| <div class="carousel-item"> | ||
| <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Third slide"> | ||
| <img class="d-block w-100" src="/images/software-management-feature-slide-2-2144x960@2x.png" alt="Third slide"> |
There was a problem hiding this comment.
Replace the generic alt text on the updated carousel images.
“Second slide” and “Third slide” do not convey any meaning to screen-reader users. Use short descriptive alt text that matches the slide content, or mark these screenshots decorative if the adjacent copy already carries the full message.
Suggested alt text
- <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Second slide">
+ <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Automate patch management">
...
- <img class="d-block w-100" src="/images/software-management-feature-slide-2-2144x960@2x.png" alt="Third slide">
+ <img class="d-block w-100" src="/images/software-management-feature-slide-2-2144x960@2x.png" alt="Self-service on any OS">📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Second slide"> | |
| </div> | |
| <div class="carousel-item"> | |
| <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Third slide"> | |
| <img class="d-block w-100" src="/images/software-management-feature-slide-2-2144x960@2x.png" alt="Third slide"> | |
| <img class="d-block w-100" src="/images/software-management-feature-slide-3-2144x960@2x.png" alt="Automate patch management"> | |
| </div> | |
| <div class="carousel-item"> | |
| <img class="d-block w-100" src="/images/software-management-feature-slide-2-2144x960@2x.png" alt="Self-service on any OS"> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@website/views/pages/software-management.ejs` around lines 40 - 43, Update the
two carousel image alt attributes so they are meaningful to screen readers:
replace the generic "Second slide" and "Third slide" alt text on the <img>
elements whose src values end with
"software-management-feature-slide-3-2144x960@2x.png" and
"software-management-feature-slide-2-2144x960@2x.png" with concise descriptive
text that summarizes each slide's visible content (or use an empty alt="" if the
adjacent heading/paragraph already conveys the same information and the images
are purely decorative).
Update page to reflect Fleet's auto patch capabilities
Summary by CodeRabbit
Release Notes