Skip to content

Rest cleanup: unify groups and regions

Alex requested to merge cleanup-rest-region into master

Second part of !910 (closed)

What does this MR do?

Most of the code and the database handles work groups as regions. This moves the one endpoint (deleting groups) that is in the GroupRestController to the RegionsRestController.

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

Very

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 the regions admin page (http://localhost:18080/?page=region)
  4. Test if you can delete regions and work groups

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

Merge request reports