Skip to content

Add check to mailbox view

Alex requested to merge mailbox-safety-check into master

What does this MR do?

Adds another safety check to the mailbox to prevent (hopefully all) errors due to bad json data.

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

Should work. Can't really become worse

Links to related issues

!1306 (comment 269396155)

How to test

Steps a reviewer can take to verify that this MR does what it says it does e.g.

  1. Checkout branch locally
  2. Login as someone with a mailbox
  3. go to /?page=mailbox and see if your mails are visible, especially in the trash folder

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 Johannes

Merge request reports