Skip to content

Move region master update to rest controller

Alex requested to merge 798-region-admin-rest into master

Part of &9

What does this MR do?

Moves the last XHR function from the RegionAdmin module to the rest controller. The behavior should not change at all. The function is used on the region management page (orga).

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

Very sure

Links to related issues

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 region admin page http://localhost:18080/?page=region
  4. Select a region from the tree on the right hand side
  5. Start the update with the master update button (lower left)
  6. You should see a success notification

Checklist

  • added a test, or explain why one is not needed/possible... not necessary, nothing really changed here
  • 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