Skip to content

Commit

Permalink
Merge pull request #28 from andrewpbrett/patch-2
Browse files Browse the repository at this point in the history
Emphasize important gotcha.
  • Loading branch information
zapnap committed Mar 9, 2012
2 parents 717ea55 + 934263a commit cf29417
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions README.md
Expand Up @@ -27,8 +27,8 @@ method variant:

MyMailer.subject_email(params).deliver!

Oh, by the way. Don't forget that your async mailer jobs will be processed by
a separate worker. This means that you should resist the temptation to pass
Oh, by the way. Don't forget that **your async mailer jobs will be processed by
a separate worker**. This means that you should resist the temptation to pass
database-backed objects as parameters in your mailer and instead pass record
identifiers. Then, in your delivery method, you can look up the record from
the id and use it as needed.
Expand Down

0 comments on commit cf29417

Please sign in to comment.