-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Update documentation to reflect multiple SLA Configurations #9844
Conversation
Hi there 👋, @DryRunSecurity here, below is a summary of our analysis and findings.
Note 🟢 Risk threshold not exceeded. Tip Get answers to your security questions. Add a comment in this PR starting with @DryRunSecurity. For example...
Powered by DryRun Security |
docs/content/en/usage/features.md
Outdated
@@ -414,14 +414,15 @@ Similar Findings | |||
DefectDojo allows you to maintain your security SLAs and automatically | |||
remind teams whenever a SLA is about to get breached, or is breached. | |||
|
|||
Simply indicate in the `System Settings` for each severity, how many | |||
days teams have to remediate a finding. | |||
To use with SLAs, open the `System Settings` page and click the 'Enable Finding SLAs' checkbox. |
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.
In order to apply SLAs to findings, open System Settings
and check 'Enable Finding SLAs'.
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.
Thank you!
docs/content/en/usage/features.md
Outdated
@@ -1,4 +1,4 @@ | |||
--- | |||
f--- |
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.
remove "f"
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.
Thank you! Sorry for missing that one
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.
Approved
(assuming @manuel-sommer changes get done)
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.
Just commenting so I can commit this suggestion
The documented process for creating an SLA configuration is out of date, along with the screenshot.
These documentation changes are intended to capture how SLA configurations currently work, since the ability to work with multiple SLA configurations has been added.