-
-
Notifications
You must be signed in to change notification settings - Fork 321
Introduce EmailTemplate classes
#3632
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
base: develop
Are you sure you want to change the base?
Conversation
| * @var array<string, string> Placeholders for email templates | ||
| */ | ||
| private static array $_message_placeholders = []; | ||
| public static function send(User $recipient, EmailTemplate $emailTemplate) |
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.
Need to keep support for setting email, Such as Store and Forms module will send emails to quests based on the email they entered
core/classes/Emails/Email.php
Outdated
| ); | ||
| } | ||
|
|
||
| public static function sendRaw(int $type, User $recipient, string $subject, string $content) |
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.
Same here i guess ^^
| "api/username_updated": "Username updated successfully", | ||
| "api/webhook_added": "The webhook has been created", | ||
| "emails/change_password_message": "To reset your password, please click the following link. If you did not request this yourself, you can safely delete this email.", | ||
| "emails/change_password_subject": "Forgot password", |
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.
Breaking all languages just for term rename :/
Co-authored-by: StyleCI Bot <[email protected]>
This reverts commit 7592d7b.
No description provided.