Skip to content

Fix another email quickreply bug

Alex requested to merge fix-mail-quickreply-bug into master

What does this MR do?

Extension to !1321 (merged). Fixes the sentry error https://sentry.io/share/issue/52ae1b1c4205422e85a63f8ca37a7740/ if an email in quickreply has no 'personal' field set.

How confident are you it won't break things if deployed?

Should work.

How to test

  1. Checkout branch locally
  2. Login as someone with a mailbox
  3. Quickreply to a mail (on dashboard) and check if the reply was sent to that person

Checklist

  • added a test, or explain why one is not needed/possible...
  • 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))
  • Once your MR has been merged, you are responsible to update the #foodsharing-beta Slack channel about what has been changed here. They will test your work in different browsers, roles or other settings
Edited by Alex

Merge request reports