Skip to content

Add flash error for setAssignees failure

Scott Stern requested to merge ss/add-error-state-for-assignees into master

What does this MR do?

Add error state for setAssignees failure. The UX is currently being revised by @hollyreynolds

Repro:

  • rails c
  • Feature.enable(:swimlanes)
  • Go to a group board http://127.0.0.1:3000/groups/h5bp/-/boards?group_by=epic
  • Block the /graphql domain request to see the error
  • Open the dropdown and select an assignee
  • When its being set notice the error banner appears

Screenshots (strongly suggested)

2020-11-23_11.58.02

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 Scott Stern

Merge request reports