Skip to content

Jira Issues: Add assignees to sidebar

Tom Quirk requested to merge jira-issues-sidebar-assignees into master

What does this MR do?

Introduces a new Assignee component to use in the Jira Issues sidebar.

This is an iteration towards #298724 (closed).

This feature is behind feature flag jira_issues_show_integration.

How to test (optional)

Prerequisites

In Jira

  1. Create a Jira issue

In GitLab

  1. Set up Jira integration with GitLab: https://docs.gitlab.com/ee/user/project/integrations/jira.html
  2. Navigate to the Jira Issues section of your project: https://docs.gitlab.com/ee/user/project/integrations/jira.html#view-jira-issues
  3. Append the issue reference (i.e. AP-1) to the URL, and load the page - this will manually navigate you to the individual Jira page
  4. Observe

Screenshots (strongly suggested)

Screenshots demonstrate 3 possible states: no assignee, assignee with no avatar, and assignee with avatar.

Collapsed Expanded
Screen_Shot_2021-02-16_at_11.55.11_am Screen_Shot_2021-02-16_at_11.55.23_am
Screen_Shot_2021-02-12_at_5.11.16_pm Screen_Shot_2021-02-12_at_5.11.10_pm
Screen_Shot_2021-02-16_at_11.54.51_am Screen_Shot_2021-02-16_at_11.54.45_am

Hover state with assignee

Collapsed hover state Expanded hover state
Screen_Shot_2021-02-16_at_11.54.55_am Screen_Shot_2021-02-16_at_11.54.38_am

Hover state no assignee

Collapsed hover state Expanded hover state
Screen_Shot_2021-02-16_at_11.55.15_am No hover state for the expanded state Screen_Shot_2021-02-16_at_11.55.23_am

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 Tom Quirk

Merge request reports