Skip to content

Refactor settings model to gateway

Alex requested to merge 9-refactor-settings-model into master

Part of #9 (closed)

What does this MR do?

Extracts all SQL statements from the SettingsModel to the SettingsGateway.

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

Not completely sure. All tests pass and I can't find any problems, but some more testing would be good.

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible... tests are already there
  • 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 Johannes

Merge request reports