Skip to content

Fix typo in MR CSV export

Max Woolf requested to merge 3619-bug-email-typo into master

What does this MR do?

Whilst verifying this issue in staging, I noticed a typo.

The HTML email template for a CSV export of merge requests was displaying merge_requests instead of merge request. This did not affect the plain text email.

This was discovered in staging, but the feature is behind a feature flag so a deployment to production is not an problem.

Screenshots (strongly suggested)

Screenshot_2020-10-28_at_15.42.45

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

Related to #3619 (closed)

Edited by Max Woolf

Merge request reports