Skip to content
Snippets Groups Projects

Fix link in plain-text re-request MR review mails

What does this MR do and why?

Tiny follow-up to Added backend for requesting a new review (!52321 - merged)

In the plain-text email template merge_request_reference_link() was used, which outputs the HTML link – not just the raw URL. This causes weirdness in some email clients.

Screenshots or screen recordings

Before After
image image

How to set up and validate locally

  1. Disable multipart emails
  2. Re-request review on an MR
  3. Observe HTML output in plaintext email

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading