Skip to content

Generalize alert details status

Alexander Turinske requested to merge 321891-change-alert-status into master

What does this MR do?

Generalize alert details status

  • allow for configuration of statues in the sidebar and details table of alerts
  • ensure tracking alert status changing continues to send back the same information as before
  • ensure changing status works on both alert management page and threat monitoring page
  • ensure alert details on the alert management details page, threat monitoring alert details page, and incident alert details tabs looks good

Screenshots (strongly suggested)

Page Before After
DEMO! The demo shows that everything is working on the alert managment table page, the alert managemnt details page, the incident alert details tab, the threat monitoring alert list page, and the threat monitoring alert details page demo_alert_status_change_02
GraphQL request staging_graphql_request graphql_request
Incident Alert Details - incident_page
Operations Alert Details - operations_alert_details_after
Network Alert Details - network_policy_alert_details_after
Changing Alert Tracking Currently in staging Changing on details page Changing on operations table
They are all the same staging_tracking details_trakcing table_tracking

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #321891 (closed)

Edited by Alexander Turinske

Merge request reports