Skip to content

Fix quickreply bug in mailbox

Alex requested to merge fix-mailbox-sentry-bug into master

What does this MR do?

Fixes a bug that was reported on sentry. This allows emails being sent via quickreply if the sender has no 'personal' field.

https://sentry.io/share/issue/4dc1ff76518d45f89a054110972c9724/

https://sentry.io/share/issue/7b3dda7041b9431697815b43b2f1882f/

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

Should work

How to test

Needs a quick reply (on dashboard) to a mail sent from a non-foodsharing address without personal name set. Can probably not be easily tested locally without adding the email manually in the db.

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 Chris Oelmueller

Merge request reports