Skip to content

Fix for broken WallPost Permissions

Dominik Hayon requested to merge 352-More-fixes-for-wallpost-permissions into master

Closes (parts of) #352 (closed) & #718 (closed)

What does this MR do?

Fixes for cases where walls broke after !1353 (merged).

  1. Allows Orgas to delete all posts from foodshare poin walls
  2. Allows workgroup admins to see walls for applicants again

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

Should at least not break more walls.

Links to related issues

Any relevant links (issues, documentation, slack discussions).

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 Orga
  3. Go to fairteiler
  4. Delete some posts 1️⃣
  5. Go to region settings
  6. Make a second user admin for a workgroup
  7. Login as thirs user
  8. Apply for that workgroup
  9. Login as second user
  10. Go to applicants page for your group
  11. Be able to post to wall and see those posts 2️⃣

Screenshots (if applicable)

Any relevant screenshots if this is a design / frontend change

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 Dominik Hayon

Merge request reports

Loading