Skip to content

Allow the assignment of alerts to users

Sarah Yasonik requested to merge alert-assignee-dropdown into master

What does this MR do?

This MR adds support for assigning alerts to users through the UI from the alert management detail view.

This includes the following items:

  • Adds service to assign users to alerts
  • Exposes assignee mutation via GraphQL API
  • FE pieces - Approved & merged from !32955 (merged)

Out of scope: Multiple assignees. This is something we eventually will want to support, so this code keeps that in mind. But if multiple assignees are specified through the API, an error is returned.

Context

This MR represents a vertical slice of functionality for assigning alerts to users. This is slice 3/4.

This work will occur in 4 slices:

  1. Create sidebar view for alert - !32642 (merged)
  2. Add assignee to list view - !32609 (merged)
  3. Add assignee dropdown to sidebar of alert detail view
  4. Add system notes for alert assignment - MR TBD

Slices 2-4 will all be behind a alert_assignee feature flag. Grouping the work in this way will allow us to deliver functional chunks of work which can be validated individually.

Screenshots

Alert Management List Alert details Alert details selecting
Screenshot_from_2020-05-29_22-19-29 Screenshot_from_2020-05-29_22-22-11 Screenshot_from_2020-05-29_22-22-18

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
Edited by Rémy Coutable

Merge request reports