Skip to content
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

Clarify rules instructions #467

Closed
2 tasks
adhilto opened this issue Oct 8, 2024 · 2 comments · Fixed by #497
Closed
2 tasks

Clarify rules instructions #467

adhilto opened this issue Oct 8, 2024 · 2 comments · Fixed by #497
Assignees
Milestone

Comments

@adhilto
Copy link
Collaborator

adhilto commented Oct 8, 2024

💡 Summary

One of the instructions steps says: "From the Actions page, you can change the severity for the alert to High, Medium, or Low, send an alert to the alert center if the rule's conditions are met, set up admin email notifications, and specify recipients for those notifications."

As a user, I want to know what I need to do, rather than what I can do. I can see for myself what I can do just by looking around. What isn't immediately clear is what needs to be done to meet the baseline requirement.

Also, there's a step missing between "Select the rule by clicking the table row for the rule" and "Click Next: View Conditions." You need click "Edit rule" first. Alternatively, you can skip straight to the actions page by just clicking the "Actions" box, which would cut out two steps.

Implementation notes

The required action is:

  • Enable admin center alerts, if that option is available
  • If admin center alerts are not an available, enable email notifications.

Notably, not all rules support admin center alerts (e.g., "TLS Failure"), which is something we should note in the baseline.

Acceptance criteria

  • Instructions have been clarified with required actions
  • Instructions have been adjusted to cover the missing step
@mdueltgen
Copy link
Collaborator

Just wanted to confirm this is for the implementation steps of Common Controls 13.1?

@adhilto
Copy link
Collaborator Author

adhilto commented Nov 4, 2024

Just wanted to confirm this is for the implementation steps of Common Controls 13.1?

That's right

@mdueltgen mdueltgen self-assigned this Nov 7, 2024
@mdueltgen mdueltgen linked a pull request Nov 7, 2024 that will close this issue
14 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants