Skip to content

removed false setting of disabled bool in the notification settings

Peter Tönnies requested to merge PT-317-fix-disbaled-notification-settings into master

Closes issues0#317

What does this MR do?

The variable "disabled" was not cleared but passed to other notification settings than those for the Fair-Teiler. This MR fixes this and makes it possible to unsubscribe from forum posts again, if beeing a Fair-Teiler-responsible.

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

Sure that it will not break anything. But I will need to see it in action on beta whether it fullfills all needs.

Checklist

  • added a test, or explain why one is not needed/possible... (works in localhost)
  • 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 Chris Oelmueller

Merge request reports