password reset template, fix #199 #216
Conversation
|
Thank you for the pr, I've left you a comment regarding the changes you made, so if you could take a look at those |
|
@gigili I can't find any comment. Could you help me, where have you left it? |
|
I've left it as part of the review comments on this pr |
|
@gigili I can see on the screenshot, your comments still in pending. I can't see them until you submit the code review |
| 'emailPreview' => strip_tags($emailBody), | ||
| 'emailConfirmText' => 'Reset password', | ||
| 'emailActivationLink' => $activationLink, |
There was a problem hiding this comment.
This emalActivationLink should be replaced with the name passwordResetCode both here in the model file but also as a variable name in the email template html file as well.
Also maybe a new variable should be introduced, where the passwordResetCode would be used only to show the code below the button and than have a new one called passwordResetLink which would be used as a link on the button itself.
No description provided.