Skip to content

Add a basic placeholder field for `commented_by`

Robert May requested to merge mr-commented-by into master

What does this MR do?

Related #10294 (closed)

This adds an empty field to the MR approvals API endpoint for commented_by, which is how it will be referenced in the frontend (as seen in !38136 (merged)).

Currently it returns a blank array. I'll populate it with data in the next iteration.

Screenshots

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 Robert May

Merge request reports