-
Notifications
You must be signed in to change notification settings - Fork 10
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
[cosmetic] Kestra email format could use some cleanup #163
Comments
Is anyone working on this issue? |
@sampath24-ss no, up to you to provide a PR. We'll assign to another developer otherwise in the following days |
Hi @Ben8t, Thank you for the clarification! Before I proceed with the PR, could you kindly point me to the relevant file(s) or folder where the email rendering code is located? Specifically, I’m looking into the issue. I appreciate your guidance! |
@sampath24-ss I think you can find the HTML template here, and how it is sent here |
@Ben8t Maybe it would be a good idea to create an issue on OSS repo with the list of all email designs that need improving and with designs for it, maybe someone from the community could work on it as it is pretty simple task, but will help us greatly. |
I've identified the issue and made the necessary changes. Unfortunately, I'm unable to run the project locally due to an issue with my machine, so I haven't been able to submit the PR yet. Just wanted to keep you updated. Thank you! |
Hi, |
Thanks @sampath24-ss , can you linked it here in comment please :) ? |
Closed via #174. |
Thanks for the support @Ben8t & @MilosPaunovic |
Describe the issue
I set up an email alert and there are a couple issues:
Environment
The text was updated successfully, but these errors were encountered: