-
Notifications
You must be signed in to change notification settings - Fork 21.3k
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
[ci skip] Clarify Action Mailer/Active Job usage note #16612
Conversation
use `deliver_later` now to send the emails and when you decide to start sending the | ||
email from a background job you'll just have to setup Active Job to use a queueing | ||
NOTE: Active Job's default behavior is to execute jobs ':inline'. So, you can use | ||
`deliver_later` now to send emails, and when you later decided to start sending |
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.
it should be when you later decide
😸
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.
omg. embarrassing! fixed :)
backend (Sidekiq, Resque, etc). | ||
|
||
If you want to send the emails right away (from a cronjob for example) just | ||
call `deliver_now`: | ||
If you want to send emails right away (from a cronjob for example) just call `deliver_now`: |
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.
This is a nit-pick but could you please wrap this line around 80 chars ? It looks good to me so far, thanks for your contribution ! :-)
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.
will do! sorry about that.
[ci skip] Clarify Action Mailer/Active Job usage note
Thanks ! |
@robin850 @loganhasson Thank you! <3 <3 <3 |
No description provided.