Skip to content

Introduce very basic permissions for email newsletter sending

Johannes requested to merge Jo-introduce-newsletter-permissions into master

What does this MR do?

Introduce very basic permissions for email newsletter sending. For now this only replaces orga permissions plus newsletter workgroup admin permissions. There is a lot of refactoring needed in this class but for now this MR is only concentrating on the permissions part.

Removed the entry from the menu if permission is false. This MR would allow to remove this permission from orga members and allow to concentrate on the newsletter workgroup who knows its limitations.

This adds power to Newsletter work group (if the aren't Orga members already - which they currently are)

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

Quite sure 90%

Links to related issues

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