Fix initialization of AsyncMail
What does this MR do?
In my unified DI changes (!544 (merged)) I failed to change some AsyncMail
instantiations to have an instance of Mem
passed in.
This caused mail sending to fail (for some cases at least). I found 3 occurrences where this was missed.
How confident are you it won't break things if deployed?
Very, intellij showed me error, and is happy afterwards :)
Links to related issues
Checklist
-
added a test, or explain why one is not needed/possible... meh, this is quickfix for now, we have poor test coverage of these extra kind of services -
no unrelated changes -
asked someone for a code review -
joined #foodsharing-beta channel at https://slackin.yunity.org -
added an entry to CHANGELOG.md (description, merge request link, username(s))
Edited by Nick Sellen