[openstreetmap/openstreetmap-website] Welcome email (PR #4288)

Milan Cvetkovic notifications at github.com
Fri Oct 13 16:24:46 UTC 2023


@milan-cvetkovic commented on this pull request.



> @@ -1594,6 +1594,90 @@ en:
       greeting: "Hi,"
       hopefully_you: "Someone (hopefully you) would like to change their email address over at %{server_url} to %{new_address}."
       click_the_link: "If this is you, please click the link below to confirm the change."
+    welcome_email:
+      subject: "[OpenStreetMap] Welcome!"

I am not really sure how to reduce the duplication of messages for welcome email welcome screen.

It looks like the template files need to be stored in specific directories: `views/user_mailer/welcome_email.*.erb` and `views/site/welcome.html.erb` - so we cannot reuse the same template file.

I could potentially reuse the strings - use `layouts/site/welcome` strings instead of `layouts/user_mailer/welcome_email` in `views/user_mailer/welcome_email.*.erb`, but that didn't seem to be clean.


-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4288#pullrequestreview-1676847852
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4288/review/1676847852 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231013/ce1fcbbc/attachment.htm>


More information about the rails-dev mailing list