-
Notifications
You must be signed in to change notification settings - Fork 195
[Security] Privmon – AD integration groups update #4528
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
Vale Linting ResultsSummary: 2 suggestions found 💡 Suggestions (2)
|
🔍 Preview links for changed docs |
benironside
left a comment
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.
Great work, copy looks perfect to me. The only things I would suggest is moving the content from line 50 to line 46, so it's not in the switcher, I think that would give a little more context about this big list. I'd also consider using a stepper for the two steps on line 44 and 86. I know it's only two steps, but after the list I found myself a bit disoriented about where in the process I was.
That content actually applies only to 9.3/serverless, so I don't think moving it out of the switcher would work in this case.
Good idea! 💡 I've added a stepper now, and it definitely adds more visual clarity. 👍 |
CAWilson94
left a comment
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.
Thanks for this! Looking good 🚀
Summary
Resolves #4503:
Preview
Set up and manage privileged user monitoring > Add a supported integration
Generative AI disclosure
Tool(s) and model(s) used:
Cursor, claude-4.5-sonnet