Skip to content

Add the MR author to MR's list on the compliance dashboard

Problem to solve

We currently don't note who created an MR on the compliance dashboard but we do show the approvers.

Intended users

Further details

Not showing the author means that auditors need to click through to an MR to see who created it if they have any questions or need further details. This makes their job that little bit harder as it may involve multiple MR's and a bit more headspace to remember who did what.

Proposal

Add a created by: label underneath the MR's title with the author's name and link to their profile

Design

Question: Should we show the authors

TBD
image

Full spec to be added.

Technical implementation plan

backend changes

  1. Add :author to MergeRequestComplianceEntity e.g. expose :author, using: API::Entities::UserBasic
  2. Add :author to includes merge request attributes for compliance in ee/spec/serializers/merge_request_compliance_entity_spec.rb

frontend changes

  1. Add new createdBy: s__('ComplianceDashboard|created by:'), label to $options.strings in ee/app/assets/javascripts/compliance_dashboard/components/dashboard.vue
  2. Add the label after the issuable_reference and show the author in the same way we show the approver(s)
  3. Update ee/spec/frontend/compliance_dashboard/components/dashboard_spec.js to include a test to prove the author is shown when passed along

This could be done in one MR

Edited by Robert Hunt