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
feat: Support for non-attachment mails #6370
Conversation
7cf5e56
to
61ff368
Compare
@uds5501 @iamareebjamal This is ready for a review |
Codecov Report
@@ Coverage Diff @@
## development #6370 +/- ##
===============================================
- Coverage 65.33% 65.29% -0.04%
===============================================
Files 287 287
Lines 14765 14775 +10
===============================================
+ Hits 9646 9647 +1
- Misses 5119 5128 +9
Continue to review full report at Codecov.
|
app/api/helpers/tasks.py
Outdated
message.attach(name=attachment) | ||
mailer.send(message) | ||
logging.info('Message sent via SMTP') | ||
|
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.
blank line contains whitespace
a9178f2
to
cdda896
Compare
Shifted logic to outer try..except Supported for SMTP protocol
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.
Looks LGTM
Fixes #6047
Fixes #6327
Short description of what this resolves:
This PR aims to enable the mail task to send emails even when attachments are blocked by the mail server.
Changes proposed in this pull request: