-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃殌 Feature: Improve email templates management #6833
Comments
Me and @mirkobrombin can work on this if it's an accepted feature It may be good to work on this based on #6495 to avoid overlap/conflicts |
I'd like to see this as well. This can indeed remove some code duplicacy from the templates. |
I'm sorry but I don't understand the In the task I would also like to add a wysiwyg editor like Quill (super light) and the possibility to preview the formatted template without needing to call an API to request a password reset (for example). Maybe also add a field text where to insert an email target for sending a test email for a given template? |
@pietrodicaprio Here's a very low level basic idea: This makes the UI less cluttered. The idea is that you select either the Header, Footer or a Template & the editor opens on the side for that [I might as well make a new Feature Request for this lol]. |
Understood, I like it! Well, as you're a member of the org, how does it work exactly in terms of task execution? Should I just start working on it and open a PR that will eventually be reviewed somewhen or should I wait some confirmation before proceeding? |
@pietrodicaprio A core member will eventually check out the issue, provide feedback if this is something they can move ahead with & assign you the task. |
can i work on this? |
馃敄 Feature description
Right now the email template feature allows the user to replace the body of the global email template:
https://github.com/appwrite/appwrite/blob/main/app/config/locale/templates/email-base.tpl#L59
as (kind of) explained here.
Having the head, CSS and footer fixed in the template it makes hard to keep all the templates aligned overtime if you want to put some extra CSS in order to properly brand the templates because is required to put that CSS in every single template.
馃帳 Pitch
It would be better to allow to customize the head and the footer in separate sections and prepend/append to the body that is handled like it is right now. In this way it's easy to fix/update the CSS of all the templates in once.
馃憖 Have you spent some time to check if this issue has been raised before?
馃彚 Have you read the Code of Conduct?
The text was updated successfully, but these errors were encountered: