Skip to content

Fix email bug in forum quickreply

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

What does this MR do?

Closes #794 (closed)

Fixes a bug in the quickreply function for forum topics that I added during refactoring.

https://sentry.io/share/issue/12ef67aa41fe44bcaea37f16845cc1f9/

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

Quite sure, there was only an index missing.

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 foodsaver (1)

  3. Make sure you follow a forum topic

  4. Login as another foodsaver (2)

  5. Post something in the same topic

  6. Login as the first foodsaver (1) again

  7. Use the quickreply to answer that post

  8. Check if an email is sent foodsaver (2)

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