-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix copy in email templates #2801
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2801 +/- ##
=======================================
Coverage 26.86% 26.86%
=======================================
Files 89 89
Lines 17596 17596
=======================================
Hits 4727 4727
Misses 12183 12183
Partials 686 686 Continue to review full report at Codecov.
|
@@ -2,7 +2,7 @@ | |||
<html> | |||
<head> | |||
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> | |||
<title>{{.Username}}, please verify your e-mail address</title> | |||
<title>{{.Username}}, please verify your email address</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this change is needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have changed it to email
, because in the mail body it is email
.
Please click the following link to verify your email address within
.
If still want to revert back, i will roll it back to e-mail
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better as it was before
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, no problem.
LGTM |
LGTM, although these should eventually be moved to locales, right? |
@ethantkoenig I think yes |
Fixes #2784