Skip to content

Always show target url on note email

Andrew Fontaine requested to merge afontaine/add-link-to-notes-notification into master

What does this MR do and why?

Previously, only discussions (that is, notes that were replying to other notes) had a link to the discussion at the top of the email.

This should probably be consistent, as it also matches the HTML version better.

Changelog: changed

Screenshots or screen recordings

Old New
image image

How to set up and validate locally

  1. Comment on an issue such that it would notify them (pinging a user usually works)
  2. Open /rails/letter_opener
  3. Select your notification email
  4. Select View plain text version in the top right

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