Skip to content

Remove activity model

Alex requested to merge 9-activity-model-to-gateway into master

Part of #9 (closed)

What does this MR do?

Removes the ActivityModel class. It's functions directly access the ActivityGateway, which is why I moved them to the corresponding Xhr class. In a next step I would like to refactor that into Rest endpoints.

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

~ 99%

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 foodsaver
  3. See if all updates on the dashboard are visible

Checklist

  • added a test, or explain why one is not needed/possible... still covered by existing tests.
  • 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