Skip to content

Fix: notify locale on approved merge request email

What does this MR do and why?

Reason

Right now the email template for approved merge request notification can only be sent in English

Result

in this MR I updated the template with i18n enabled strings instead of hardcoded strings. After this MR it will be able to send languages the user prefers.

/cc @prajnamas @orozot @qk44077907

How to test locally

  1. Start your GDK instance locally or via gitpod or other ways
  2. Open the browser with your instance with path /rails/mailers
  3. Find the email template with cmd/ctrl + f search for text approved_merge_request
  4. Open that link to see the previews.

Screenshots

CleanShot_2022-06-22_at_16.47.09_2x

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

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

Edited by Wu Jeremy

Merge request reports