-
Notifications
You must be signed in to change notification settings - Fork 639
Rebuild self-service article to reflect 4.69 changes, add self-service uninstall highlight to release notes #30030
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
articles/fleet-4.69.0.md
Outdated
@@ -30,6 +30,10 @@ Fleet now supports pulling user data—like IdP email, full name, and groups—f | |||
|
|||
When a vulnerability has no primary CVSS score in the [National Vulnerability Database (NVD)](https://nvd.nist.gov/), Fleet now shows the secondary score instead. This gives Security Engineers better visibility into potential risk and helps prioritize remediation. | |||
|
|||
### Self-service software: uninstall | |||
|
|||
End users can now uninstall self-service software via the My device UI, in addition to being able to [install software](https://fleetdm.com/guides/software-self-service#installing-self-service-software-packages) from there. Software uninstalls initiated this way are tracked in Fleet's audit log. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iansltx Do we need to call out the install software bit? That additional feels clunky.
" End users can now uninstall self-service software via the My device UI. Software uninstalls initiated this way are tracked in Fleet's audit log." reads a lot more clearly.
Alternately, if we want to keep the 'in addition' text it feels more natural in front:
"In addition to being able to install self-service software via the My device UI, end users can now also uninstall self-service software.Software uninstalls initiated this way are tracked in Fleet's audit log."
Thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Main reason for including was we link to the self-service install documentation, and don't have uninstall docs.
If we want to drop to just mentioning uninstall, I should update the self-service docs as part of this PR to include a paragraph or two on that action. Shouldn't take long, so sounds like I should?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we want to drop to just mentioning uninstall, I should update the self-service docs as part of this PR to include a paragraph or two on that action. Shouldn't take long, so sounds like I should?
Yeah. I feel like we should def update the self-service docs and then we can link to those docs from here. I imagine that will be right under the install docs so people will be aware. Since this is release notes, I think it's nice to be more direct on what's being released. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool. Will ping you for review again when this is ready, net of self-service guide updates.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...and ready. Self-service article was actually pretty out of date, so revamped that quite a bit to take the frontend rebuild into account.
articles/software-self-service.md
Outdated
|
||
## Step-by-step instructions | ||
1. Click “Software” in the main navigation menu. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@iansltx I believe that the company style guide has us writing 'Select' instead of 'Click' throughout.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got it. Was reusing existing text but will swap it since we're already changing all the things.
@eugkuo Got another revision for you. Marked existing feedback as resolved mainly because content has changed out from under it 2x at this point so it's harder to track what still needs changing. |
…dated references in software self-service article
de58a4e
to
078931c
Compare
No description provided.